mirror of
https://github.com/bitcoin/bitcoin.git
synced 2025-03-05 14:06:27 -05:00
Merge #15173: [doc] explain what .python-version does
04215eb328
[doc] explain what .python-version does (Sjors Provoost)
Pull request description:
Documentation followup to https://github.com/bitcoin/bitcoin/pull/14884#discussion_r241444376.
Tree-SHA512: 7b2b2718f998b7257273fab2e4f1d127f7af468cd26b2bf05b3dc7d6367e349e0a9c92e976a2760e83cf56feb68c37c89cb3a8d92e20be900bea3fb7f1f3d47b
This commit is contained in:
commit
391a27376b
1 changed files with 4 additions and 0 deletions
|
@ -20,6 +20,10 @@ don't have test cases for.
|
||||||
- Where possible, try to adhere to [PEP-8 guidelines](https://www.python.org/dev/peps/pep-0008/)
|
- Where possible, try to adhere to [PEP-8 guidelines](https://www.python.org/dev/peps/pep-0008/)
|
||||||
- Use a python linter like flake8 before submitting PRs to catch common style
|
- Use a python linter like flake8 before submitting PRs to catch common style
|
||||||
nits (eg trailing whitespace, unused imports, etc)
|
nits (eg trailing whitespace, unused imports, etc)
|
||||||
|
- The oldest supported Python version is specified in [doc/dependencies.md](/doc/dependencies.md).
|
||||||
|
Consider using [pyenv](https://github.com/pyenv/pyenv), which checks [.python-version](/.python-version),
|
||||||
|
to prevent accidentally introducing modern syntax from an unsupported Python version.
|
||||||
|
The Travis linter also checks this, but [possibly not in all cases](https://github.com/bitcoin/bitcoin/pull/14884#discussion_r239585126).
|
||||||
- See [the python lint script](/test/lint/lint-python.sh) that checks for violations that
|
- See [the python lint script](/test/lint/lint-python.sh) that checks for violations that
|
||||||
could lead to bugs and issues in the test code.
|
could lead to bugs and issues in the test code.
|
||||||
- Avoid wildcard imports where possible
|
- Avoid wildcard imports where possible
|
||||||
|
|
Loading…
Add table
Reference in a new issue