hagopj13 / hagopj13/node-express-boilerplate
Style proposal: async/await vs .then()
- 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
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