Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm making progress on #951 again but have hit a tricky wall. The code style is inconsistent and I don't know what style to follow. The project should adopt a formatting tool to enforce a consistent style to simplify external contributions. A possible solution is to adopt prettier.
The prettier config that best matches the current style is this:
Another possible option to enable is
arrowParens
to eitheralways
require them or toavoid
them. I'd recommend sticking to the default (always
) but I know the current style sometimes avoids them.This PR adds prettier, the prettier config, and the
format
npm script. I did not commit the changes resulting from running prettier to keep it focused. I can do that in a separate PR or you can just run the format script yourself.