Fix the bad rebase
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Sep 30 2021
Poor rebase
Rebase
Rebase
unrelated failure
Sep 29 2021
Rebase on top of D10216 and expand upon what changed in the tests in the summary.
Hopefully this will make more explicit what are the changes to this hex mess, and
give confidence that the tests are correct and covering the relevant cases.
@bytesofman see D10214 follow-on Diff
add reviewer
Set the minimum isort version to 5.6.4, which seems to be a stable version used by debian. 5.0.0 produces different results wrt to newlines before the first import compared to the latest version (5.9.3), whereas 5.6.4 produces identical results when run on our code.
make install-dependencies.sh install isort for the CI.
I see. I saw only the inline comments. Sorry, I will pay closer attention next time.
You missed that part:
In D10124#229719, @Fabien wrote:Please address the remaining items before submitting for review
I don't understand what you mean.
Please address the remaining items before submitting for review
I missed an item in CONTRIBUTING.md: include isort in the paragraph that explains how to install mypy with pip install on debian.
Please address the remaining items before submitting for review
This can be reviewed and landed independently of D10210.
address review: add url, improve description and installation instruction, remove unused methods, set parameters in getMandatoryFlags
rebase to include recently added wallet_startup.py file