Skip to content
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

CI(project): and CI to fix spell #39

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

tpoisonooo
Copy link
Contributor

typo 太多了,估计有几百个..

  • 加了个 CI 自动查 doc 的 typo
  • CI 对 python 代码做 isort 和 yapf
  • 别的 .h .c .cpp 里的 typo 就不敢动了

手动触发 pre-commit

$ pip install pre-commit
$ cd megcc
$ pre-commit install .
$ pre-commit run --all-files

可修改 .pre-commit-config.yaml,去掉 --skip 看看实际有多少 typo ..

@yeasoon
Copy link
Collaborator

yeasoon commented Feb 1, 2023

LGTM, @haolongzhangm please help to review

@chenqy4933
Copy link
Collaborator

LGTM

@chenqy4933
Copy link
Collaborator

@yeasoon 有时间统一修一下吧,“可修改 .pre-commit-config.yaml,去掉 --skip 看看实际有多少 typo ..”

@yeasoon yeasoon mentioned this pull request Feb 1, 2023
@yeasoon yeasoon merged commit cc7cc8e into MegEngine:main Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants