While it's always great to see performance gains, my largest pain point with prettier was never performance. Instead my only gripe with prettier is the "line wrapping noise" it creates, illustrated here by Anthony Fu: https://antfu.me/posts/why-not-prettier#the-line-wrapping-no... Would it be realistic to expect a solution for this issue now that "prettier needs to step up it's game"?
Specifically for reviewing a pull request in GitHub, wouldn't the "Hide whitespace" setting reduce some of this noise? I could be mistaken, though, but that's how I interpreted that setting. 0: https://github.blog/2011-10-21-github-secrets/
This is often useful, but JavaScript specifically has the annoying property that newlines can be semantically meaningful.
For example, if someone changes:
function isUserBanned(username) {
return db.findUserByName(username)?.banned;
}
function isUserBanned(username) {
return
db.findUserByName(username)?.banned;
}