feat: refactor linter into smaller modules

conda-forge / conda-smithy

The tool for managing conda-forge feedstocks.

https://conda-forge.org/

BSD 3-Clause "New" or "Revised" License

147 stars 177 forks source link

feat: refactor linter into smaller modules #1981

Closed nichmor closed 1 month ago

nichmor commented 1 month ago

Checklist

[ ] Added a news entry
[ ] Regenerated schema JSON if schema altered (python conda_smithy/schema.py)

Aims to make #1970 to make easier to review and implement by splitting the linter into smaller modules.

nichmor commented 1 month ago

hey @beckermr ! This PR aims to split linter into smaller modules ( like, linter.py, lints.pyand hints.py)

wolfv commented 1 month ago

@beckermr I am currently sitting next to @nichmor and would be happy to do a deep walkthrough through the PR with him. If you are OK, I would merge it after my review.

beckermr commented 1 month ago

Sure go for it.

wolfv commented 1 month ago

Two small comments, rest LGTM :)

jaimergp commented 1 month ago

Don't forget to include the news/ file in the next ones (you can put the one for this PR later) 🙏

nichmor commented 1 month ago

Don't forget to include the news/ file in the next ones (you can put the one for this PR later) 🙏

my fault! Added it here: https://github.com/conda-forge/conda-smithy/pull/1982