search

mbarkhau / pylint-ignore

MIT License
14 stars 8 forks source link

Add pre-commit metadata to the repository #10

Closed jamesquilty closed 2 years ago

jamesquilty commented 2 years ago

Adds a .pre-commit-hooks.yaml file and updates the README.md file, as requested in #9.

mbarkhau commented 2 years ago

Just FYI I'm aware of this and thank you for the effort. 💯

I hopefully will have time to review over the weekend.

mbarkhau commented 2 years ago

I'm going to see if I can reduce the scope of this PR to something that moves us forward and is useful to some people. If you have an idea in the future about how to handle your issues, you're welcome to create another issue.

jamesquilty commented 2 years ago

Thanks for the attention on this. I think that removing the proposed pylint-ignore-update from .pre-commit-hooks.yaml, and removing the references in the README to procedures involving that hook would achieve the goal of reducing the scope and progressing this PR to merge-able state.

The problems we're encountering are really pylint configuration/environment problems.