Skip to content

Add the config file option and tests #80

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

rdimaio
Copy link

@rdimaio rdimaio commented Dec 20, 2024

Follow-up from #19, adding:

  • fixing the merge conflict
  • fixing the test added in the original PR
  • added a couple more tests

@rdimaio
Copy link
Author

rdimaio commented Dec 20, 2024

Note: the ill-formed and valid config tests are based on the ones in the main repo: https://github.com/codespell-project/codespell/blob/4aca7ad0ea45b1540ec2046db32252647729ac75/codespell_lib/tests/test_basic.py#L1229-L1276

@spetrosi
Copy link

spetrosi commented Feb 7, 2025

Can we merge this? That's an essential fix

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.

3 participants