Merge #15249: Docs: Update python docs to reflect that wildcard imports are disallowed
f618c58b75
Docs: Update python docs to reflect that wildcard imports are disallowed (Ben Woosley)
Pull request description:
These have been disallowed via flake8 since: #13054
Tree-SHA512: f41651fd883e3786a7e87c4aa4c54749308e576287f2f88da3f1d8d0f59e14519d99061f1efd05b8745f495f87a14286cea576f1507d10ccd226f8cf2f2b3cc8
This commit is contained in:
commit
ab46fe6ec1
2 changed files with 2 additions and 2 deletions
|
@ -26,7 +26,7 @@ don't have test cases for.
|
||||||
The Travis linter also checks this, but [possibly not in all cases](https://github.com/bitcoin/bitcoin/pull/14884#discussion_r239585126).
|
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
|
||||||
- Use a module-level docstring to describe what the test is testing, and how it
|
- Use a module-level docstring to describe what the test is testing, and how it
|
||||||
is testing it.
|
is testing it.
|
||||||
- When subclassing the BitcoinTestFramwork, place overrides for the
|
- When subclassing the BitcoinTestFramwork, place overrides for the
|
||||||
|
|
|
@ -13,7 +13,7 @@ is testing and *how* it's being tested
|
||||||
# libraries then local imports).
|
# libraries then local imports).
|
||||||
from collections import defaultdict
|
from collections import defaultdict
|
||||||
|
|
||||||
# Avoid wildcard * imports if possible
|
# Avoid wildcard * imports
|
||||||
from test_framework.blocktools import (create_block, create_coinbase)
|
from test_framework.blocktools import (create_block, create_coinbase)
|
||||||
from test_framework.messages import CInv
|
from test_framework.messages import CInv
|
||||||
from test_framework.mininode import (
|
from test_framework.mininode import (
|
||||||
|
|
Loading…
Reference in a new issue