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

checks: Add Ruff per-file-file-ignores section in pyproject.toml #1320

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

echoix
Copy link
Member

@echoix echoix commented Feb 16, 2025

Ignoring DOC502 in i.sam2, as it wasn't discussed what convention to use in docstrings (list the exceptions that the function explicitly raises itself, or include exceptions that can be thrown by all called functions). We should let the grass-addons be more free to accept existing code (even though this is new code), whatever the convention was, in order to lower the barrier of entry. It was the first introduction of that error yet.

The pyproject.toml file was also reformatted (including sorting of keys), that I was constantly reverting since configuring ruff here, except this time where I let it through.

@echoix echoix merged commit 5aafa6c into OSGeo:grass8 Feb 16, 2025
7 checks passed
@echoix echoix deleted the ruff-per-file-ignore-DOC502 branch February 16, 2025 14:52
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.

1 participant