-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Closed
Labels
locked-due-to-inactivityPlease open a new issue and fill out the template instead of commenting.Please open a new issue and fill out the template instead of commenting.status:has prIssues with an accompanying pull request. These issues will probably be fixed soon!Issues with an accompanying pull request. These issues will probably be fixed soon!
Description
Trying to understand how this:
const somePromise = promise.resolve().then(console.log).catch(err => {
console.log(err)
return null
})is considered more readable than this:
const somePromise = promise
.resolve()
.then(console.log)
.catch(err => {
console.log(err)
return null
});Doesn't feel like it matches up with how clean code should be (based on the Clean Code and Code Complete books)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
locked-due-to-inactivityPlease open a new issue and fill out the template instead of commenting.Please open a new issue and fill out the template instead of commenting.status:has prIssues with an accompanying pull request. These issues will probably be fixed soon!Issues with an accompanying pull request. These issues will probably be fixed soon!