-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Drop pylint #10635
Copy link
Copy link
Closed
Labels
P2Medium priority, add to the next sprint if no P1 availableMedium priority, add to the next sprint if no P1 available
Description
We have decided to drop pylint from Haystack CI.
Why?
- Nowadays, most rules can be covered by ruff, which we already use
- pylint is slow
- Adds overhead: ruff is run frequently (manually or via pre-commit), while pylint is often forgotten, leading to PR failures and lost time.
What to do?
- configure ruff to cover as many as possible of the pylint rules we use
- drop pylint and pylint ignore comments
- apply the same changes to Haystack Experimental (Core Integrations never used pylint)
👋 Hello there! This issue will be handled internally and isn't open for external contributions. If you'd like to contribute, please take a look at issues labeled contributions welcome or good first issue. We'd really appreciate it!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P2Medium priority, add to the next sprint if no P1 availableMedium priority, add to the next sprint if no P1 available