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
Add hook: recipe linting #141
Conversation
Should we include it to Conan Center? As linter was embedded in Conan, all recipes were checked by pylint before. |
It can be useful for Conan Center too, of course. |
Please, don't forget to add it in README file :) |
import astroid # Conan 'pylint_plugin.py' uses astroid | ||
from pylint import epylint as lint | ||
except ImportError as e: | ||
sys.stderr.write("Install pylint to use 'recipe_linter' hook: 'pip install pylint astroid'") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please consider adding some nice comments in the escaping chars line and the ANSI one for clarification |
Add hook to lint recipes. It should substitute Conan built-in (conan-io/conan#6152)
Not to be merged before conan-io/conan#6152, it might require additional requirements in this repo.
test_dynamic_fields
should only run with theconans.pylint_plugin.py
with the python_requires added.