-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[Enhancement] Add codespell pre-commit hook and fix typos #920
Conversation
Codecov Report
@@ Coverage Diff @@
## master #920 +/- ##
==========================================
- Coverage 89.90% 89.85% -0.05%
==========================================
Files 117 118 +1
Lines 6555 6558 +3
Branches 1019 1019
==========================================
Hits 5893 5893
- Misses 461 464 +3
Partials 201 201
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
demo/MMSegmentation_Tutorial.ipynb
Outdated
@@ -48,9 +48,9 @@ | |||
}, | |||
"outputs": [], | |||
"source": [ |
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.
We may ignore .ipynb
.
…#920) * add codespell pre-commit hook and fix typos * Update mmseg/models/decode_heads/dpt_head.py * Update mmseg/models/backbones/vit.py * Update mmseg/models/backbones/vit.py * fix typos * skip formating typo * deprecate formating * skip ipynb * unstage ipynb changes * unstage ipynb changes * fix typos in ipynb * unstage ipynb changes
* add cfg * add cfg * update configs * add ochuman benchmark * update format
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)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
It may break CI if new PRs have typos.