hagopj13 / hagopj13/node-express-boilerplate

Style proposal: async/await vs .then()

Open
#103 8 comments 6 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
7.7k
Forks
2.3k
PR merge metrics
No merged PRs in 30d

Description

I've seen there's a mix in the code where sometimes `async/await` is used and sometimes `somePromise.then().catch()`. I'd like to propose to homogenize the code by using *always* `async/await`, and I offer myself to make the PR should the proposal be accepted.

Also, if it's accepted, it should be added to the contribution guideline.

Contributor guide

Open the contributing guide

Research direction

The issue does not identify specific files or tests. First review the existing async/await and .then().catch() usage across the codebase, then check the contribution guideline. Done means agreeing on a convention, applying it consistently, and documenting the accepted style.

Written by the indexing model from the issue text.

Assessment

Tech stack
express, javascript, node.js
Domain
backend, documentation
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.