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

Fix typos in the docs using codespell #7285

Merged
merged 5 commits into from Mar 31, 2022
Merged

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 30, 2022

Description

Fix typos in the docs using https://pypi.org/project/codespell

codespell -w *.rst
codespell -w docs

Related resources

  • #...
  • #...

Checklist

  • I have opened this pull request against develop
  • I have added or modified the tests when changing logic
  • I have followed the conventional commits guidelines to add meaningful information into the changelog
  • I have read the contribution guidelines and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

Copy link
Member

@marksweb marksweb left a comment

This seems like a neat tool @cclauss 👌

Do you know if it can integrate with pre-commit hooks or github actions?

@crydotsnake
Copy link
Contributor

@crydotsnake crydotsnake commented Mar 31, 2022

Do you know if it can integrate with pre-commit hooks or github actions?

I believe it can be!: pypa/pip#7689

@cclauss
Copy link
Contributor Author

@cclauss cclauss commented Mar 31, 2022

I put it in GitHub Actions in this PR but it can also be put into pre-commit.

@marksweb
Copy link
Member

@marksweb marksweb commented Mar 31, 2022

@cclauss thanks again. I'll merge this as it's a good step forward - but I do like the sound of pre-commit handling this also.

@marksweb marksweb merged commit 7a497df into django-cms:develop Mar 31, 2022
41 checks passed
@cclauss cclauss deleted the codespell-docs branch Apr 1, 2022
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