-
Notifications
You must be signed in to change notification settings - Fork 995
Closed
Labels
CommunityIssue/PR opened by the open-source communityIssue/PR opened by the open-source community
Description
Context
I noticed that pyproject.toml of Kedro has ~30 lines of ruff configuration.
It might be beneficial to move them to a dedicated file, ruff.toml, to:
- Simplify version control and PR reviews (changes to linter config would not be touching the manifest file).
- Make all
ruffsettings be together and thus editing them would be easier. - Make the
pyproject.tomlfile smaller, as in OSS project it anyway carries a lot of responsibility and complexity because of metadata and packaging specifications.
Ruff docs for reference here.
Important
I mean the change in Kedro itself, for the benefit of maintainers. Not in starters. So users of Kedro would actually not be affected at all. Whether it's valuable in starters can be a separate discussion/issue.
Possible implementation
A PR as a simple as take ruff config sections out of pyproject.toml and add exactly them to ruff.toml.
I can help with that if that's a change the maintainers team wants. Let me know :)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
CommunityIssue/PR opened by the open-source communityIssue/PR opened by the open-source community
Type
Projects
Status
Done