jazzband / django-oauth-toolkit

OAuth2 goodies for the Djangonauts!
https://django-oauth-toolkit.readthedocs.io
Other
3.06k stars 777 forks source link

Add codespell support: config + workflow to catch new typos, let it fix some #1392

Closed yarikoptic closed 1 month ago

yarikoptic commented 5 months ago

Description of the Change

https://github.com/codespell-project/codespell is great to catch typos

Let me know if needs changelog entry etc

Checklist

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.56%. Comparing base (560f84d) to head (d77c471). Report is 5 commits behind head on master.

:exclamation: Current head d77c471 differs from pull request most recent head 0ca5eee. Consider uploading reports for the commit 0ca5eee to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1392 +/- ## ======================================= Coverage 97.56% 97.56% ======================================= Files 32 32 Lines 2132 2132 ======================================= Hits 2080 2080 Misses 52 52 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

n2ygk commented 4 months ago

@yarikoptic is this still a draft?

yarikoptic commented 4 months ago

I moved it into draft since was experimenting with fixing that typo in a url in a test -- tests pass so it is great -- we can just fix the test. Also we verified that pre-commit catches typos -- I will remove dedicated workflow. I will rebase/squash that change now to clean up history a little... force pushed -- should be all good and neat now ;)