-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve developer experience with consistent formatting #41
Conversation
- Add format command for consistency with lint - Add spell checker recommendation - Specify consistent tab width of 3 spaces
- Organize all imports by search&replace the same string in all files - Run 'format' on repository
b210d27
to
4626288
Compare
Unit Test Results 3 files ±0 30 suites ±0 2m 21s ⏱️ -6s Results for commit ad9ee17. ± Comparison against base commit 9afb1c3. This pull request removes 1 and adds 1 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked at the changes, most at the config files. I only noticed one thing where Chevrotain and Chevrotrain are used. Should be the first one I think.
36bf635
to
ad9ee17
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Add format command for consistency with lint
Add spell checker recommendation
Specify consistent tab width of 3 spaces
Fix typos and markdown formatting issues
Apply consistent format on all files and organize imports
-- Organize all imports by search&replace the same string in all files
-- Run 'format' on repository