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 more python tests #906
Conversation
This pull request introduces 3 alerts and fixes 1 when merging eaa234c into 002a662 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 2 alerts and fixes 1 when merging 85330b0 into 002a662 - view on LGTM.com new alerts:
fixed alerts:
|
85330b0
to
455bde9
Compare
This pull request introduces 2 alerts and fixes 1 when merging 909f886 into c0c734b - view on LGTM.com new alerts:
fixed alerts:
|
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.
Ok. Just no more changes in formatting please. Especially in untouched files
@@ -129,7 +129,7 @@ std::pair<uint256,uint256> CHDMintWallet::RegenerateMintPoolEntry(CWalletDB& wal | |||
/** | |||
* Generate the mintpool for the current master seed. | |||
* | |||
* only runs if the current mintpool is exhausted and we need new mints (ie. the next mint to |
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.
Fixing formatting in unrelated files is evil
PR intention
Add tests from the upper version of bitcoin for functionality that we lack tests.
Code changes brief
SyncWithChain
to avoid hanging when try to encrypt wallet immediately after startup.