-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add Checkspell CI job #5888
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
Add Checkspell CI job #5888
Conversation
tjzel
left a comment
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.
![]()
tomekzaw
left a comment
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.
Left some minor suggestions in the comments
tomekzaw
left a comment
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.
Thanks!
## Summary #5888 introduced spell checker on our CI and it turns out that there are some typos in web LA codebase. This PR aims to remove them. Also I've decided to add "lightspeed", "easings" and "snapshotting" into correct words list, let me know if I should replace them with something else. Here you can find that these forms are actually correct: - [snapshotting](https://www.verbix.com/webverbix/english/snapshot) - [easings](https://www.verbix.com/webverbix/english/snapshot) - [lightspeed](https://en.wiktionary.org/wiki/lightspeed) ## Test plan Check that CI passes.
Summary
This PR includes Checkspell CI job to repo.
Test plan
Check CI jobs... 👉👈