-
Notifications
You must be signed in to change notification settings - Fork 101
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
Lint model/revenus #1139
Lint model/revenus #1139
Conversation
653dcb3
to
884525e
Compare
setup.py
Outdated
'flake8 == 3.4.1', | ||
'flake8 >= 3.5.0, < 3.6.0', | ||
'autopep8 >= 1.4.0, < 1.5.0', | ||
'pycodestyle < 2.4.0', |
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.
pycodestyle
est déjà une dépendance de flake8
, pourquoi préciser la version explicitement, alors que l'on ne l'utilise pas directement?
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.
Parce que si t'installes fait :
pip install pycodestyle
et après
make install
la version ne sera pas downgradée. Et flake8
est incompatible avec > 2.5.0
afcf61d
to
93422ea
Compare
93422ea
to
a298e0c
Compare
Relates to #1057
Relates to #1060
Child of #372
openfisca_france/model/revenus/**/*
.Quelques conseils à prendre en compte :
Documentez votre contribution avec des références législatives.setup.py
.CHANGELOG.md
.