Skip to content

Decide on linting rules to enable for project #323

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
machow opened this issue Apr 1, 2025 · 1 comment
Open

Decide on linting rules to enable for project #323

machow opened this issue Apr 1, 2025 · 1 comment

Comments

@machow
Copy link
Collaborator

machow commented Apr 1, 2025

While reviewing #298, we briefly discussed linter warnings for the use of error messages. It seems like this category of linting might have been enabled in the ruff select config in our pyproject.toml, since they are "EM".

I'm on board with whatever linting rules seem useful to folks, and didn't realize this category existed until today! Wanted to open this issue to move discussion on the rules here. I personally have never used the "EM" rules, but this is more out of ignorance 😅 .

@nathanjmcdougall
Copy link
Contributor

I don't think the EM rules are enabled - in that case you linked it's just I personally follow that convention manually as a habit. You could choose to enable them but they're not enabled at the moment.

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

No branches or pull requests

2 participants