- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Oct 3 2019
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Add header guard comment
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Added to summary source of translations.
Where did these translations come from? (should be in summary)
Translations added in a separate diff. See D4208.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Should anything fail the script errors out immediately, but the user working tree may have been modified in between.
I think you should trap the error and restore the previous state to avoid confusing the user.
I'm sure you didn't run the test from the summary. Please make it clear that you are quoting core PR description.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Squashed together with this commit: https://github.com/bitcoin/bitcoin/pull/12560/commits/dfcd9f3e6abf3d53903227a085ff4cfecbfeb07f because the squashed commit's changes to behavior is prefered to the previous behavior.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Build Bitcoin-ABC / Diffs / Diff Testing started.
Build Bitcoin-ABC / Diffs / Diff Testing started.
The test plan doesn't run.
In D4191#100826, @fpelliccioni wrote:In D4191#100495, @deadalnix wrote:Part of the PR are missing.
It is a partial application of the PR, I forgot to clarify it, sorry. (summary edited).
I didn't apply the rule for built-in types, because it makes no sense:
for (int x : seq)
is preferred over
for (int const& x : seq).
Have I missed anything else?
Build Bitcoin-ABC / Diffs / Diff Testing passed.
In D4179#100740, @deadalnix wrote:The test plan is meant to actually be ran.
In D4179#100728, @deadalnix wrote:I see that you use make check-all in many tests. If you are using cmake, you should use ninja as it is faster. But right now, cmake is only a secondary option, so make sure the autotool build works.
Build Bitcoin-ABC / Diffs / Diff Testing started.
update the summary to clarify that it is a partial application of the PR.
In D4191#100495, @deadalnix wrote:Part of the PR are missing.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
updated test plan.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
Build Bitcoin-ABC / Diffs / Diff Testing started.
fixes test plan.
Oct 2 2019
Build Bitcoin-ABC / Diffs / Diff Testing passed.