-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Docs] Add code-spell pre-commit hook and fix a large mount of typos. #470
Conversation
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.
LGTM.
Codecov Report
@@ Coverage Diff @@
## master #470 +/- ##
==========================================
- Coverage 78.01% 77.97% -0.05%
==========================================
Files 101 102 +1
Lines 5617 5620 +3
Branches 923 923
==========================================
Hits 4382 4382
- Misses 1108 1111 +3
Partials 127 127
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM.
…open-mmlab#470) * Add code spell check hook * Add codespell config * Fix a lot of typos. * Add formating.py to keep compatibility.
Motivation
Refers to open-mmlab/mmdetection#6099, this hook can check all typos in the code, it can help us to avoid typos.
Modification
As the title.
BC-breaking (Optional)
It may break CI if new PRs have typos.
Use cases (Optional)
No, just a pre-commit hook.
Checklist
Before PR:
After PR: