Add prettier config #123
Add prettier config #123
Conversation
|
I did two commits one with the changes and one with just |
AndrewFerr
left a comment
AndrewFerr
left a comment
There was a problem hiding this comment.
For some reason I'm able to get a slightly smaller diff by running Prettier on the original source:
git checkout master -- src/ test/ && yarn prettier:format && git add src/ test/
You can use it if you like.
|
It should also be made clear that the code coverage test fails simply because this PR touches most source/test files, and the code coverage %age of the whole codebase is lower than the minimum per-PR coverage requirement...meaning 1) this is a false-positive and 2) code coverage should eventually be improved in general so this won't be an issue anymore! |
|
@toger5 why not use the same shared config that matrix-js-sdk uses? https://github.com/matrix-org/matrix-js-sdk/blob/develop/.prettierrc.cjs |
It does as of e7e2488 |
FTR, this was applied in |
|
This adds a prettier config.
Before this format on save changed a lot of lines.