Skip to content

chore: fix code spell errors #1431

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

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

dopry
Copy link
Member

@dopry dopry commented Jun 12, 2024

pre-commit is raising code spell errors

  • ignore AUTHORS- we don't need to spell check names Savin was raising an error
  • fix misspelling of compatibility in CHANGELOG.md
  • ignore assertIn from the python unittest library.

@dopry dopry force-pushed the fix/pre-commit-failures branch from 6122a91 to 30b7f58 Compare June 12, 2024 13:46
@n2ygk
Copy link
Contributor

n2ygk commented Jun 12, 2024

I commented out codespell in order to merge #1425. Will need to circle back here and make sure there aren't other codespell issues.

@dopry dopry force-pushed the fix/pre-commit-failures branch 2 times, most recently from 198e650 to 7420918 Compare June 12, 2024 13:55
@dopry dopry force-pushed the fix/pre-commit-failures branch from 7420918 to c8bdf0e Compare June 13, 2024 13:33
@dopry dopry requested a review from n2ygk June 13, 2024 13:33
@dopry
Copy link
Member Author

dopry commented Jun 13, 2024

@n2ygk I corrected all the code spell errors, It should be passing here, but I also wouldn't be opposed to removing it.

@n2ygk n2ygk merged commit 9a862fc into django-oauth:master Jun 13, 2024
@dopry dopry deleted the fix/pre-commit-failures branch June 13, 2024 21:51
@n2ygk n2ygk added this to the 3.0.0 milestone Aug 13, 2024
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.

2 participants