Page MenuHomePhabricator
Feed Advanced Search

Jan 14 2021

majcosta updated the summary of D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.
Jan 14 2021, 13:03
majcosta requested review of D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.
Jan 14 2021, 13:03
majcosta requested review of D8913: [backport#17696] qt: Force set nPruneSize in QSettings after the intro dialog.
Jan 14 2021, 13:02
majcosta requested changes to D8910: Remove PID file at the very end.

tests failing

Jan 14 2021, 11:58
majcosta accepted D8908: Fix a -Wunused-variable in the locale fuzzer.
Jan 14 2021, 11:34
majcosta accepted D8905: Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 11:29
majcosta retitled D8905: Moving dev dependencies to dev, removing unused deps, updates some of them from Moving dev dependencies to dev, removing unused deps to Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 11:02
majcosta requested changes to D8905: Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 04:01

Jan 13 2021

majcosta accepted D8904: [Cashtab] bump extension version to support new backend per D8885.
Jan 13 2021, 16:06
majcosta accepted D8901: [CashTab] Update react-copy-to-clipboard.
Jan 13 2021, 15:20
majcosta accepted D8902: rpc: Make rpc documentation not depend on rpc args.
Jan 13 2021, 15:08
majcosta closed D8762: [backport#16658] validation: Rename CheckInputs to CheckInputScripts.
Jan 13 2021, 14:44
majcosta committed rABC925120cac2cb: [backport#16658] validation: Rename CheckInputs to CheckInputScripts (authored by MarcoFalke <falke.marco@gmail.com>).
[backport#16658] validation: Rename CheckInputs to CheckInputScripts
Jan 13 2021, 14:44
majcosta requested review of D8762: [backport#16658] validation: Rename CheckInputs to CheckInputScripts.

it looks like D3878 and D3875 diverged from Core so that the tests don't rely on whether "CheckInputs" or "CheckInputScripts" gets printed to the debug log

Jan 13 2021, 13:58

Jan 12 2021

majcosta accepted D8884: rpc: fix broken RPCExamples for waitforblock(height).
Jan 12 2021, 18:31
majcosta accepted D8878: Replace std::to_string with locale-independent alternative.
Jan 12 2021, 15:18
majcosta accepted D8881: test: Fix restart node race.
Jan 12 2021, 13:55
majcosta accepted D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.
Jan 12 2021, 11:16
majcosta requested changes to D8878: Replace std::to_string with locale-independent alternative.
Jan 12 2021, 11:04
majcosta requested changes to D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.
Jan 12 2021, 10:32

Jan 11 2021

majcosta accepted D8876: gui: display Mapped AS in peers info window.
Jan 11 2021, 17:36
majcosta requested changes to D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.

this is PR17720 right? missing in the summary

Jan 11 2021, 17:30
majcosta accepted D8875: [CI] Add cashtab coverage report to the coverage panel.
Jan 11 2021, 17:19
majcosta accepted D8874: [CI] Generate a CashTab coverage summary report.
Jan 11 2021, 16:48
majcosta accepted D8873: [Cashtab] remove duplicate key from test mock.
Jan 11 2021, 15:19
majcosta accepted D8870: [Cashtab] extension with new pkgs, version bump.
Jan 11 2021, 15:10
majcosta accepted D8871: gui: Avoid redundant tx status updates.
Jan 11 2021, 14:43
majcosta requested changes to D8870: [Cashtab] extension with new pkgs, version bump.

tests are failing

Jan 11 2021, 13:21
majcosta accepted D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 13:20
majcosta accepted D8869: [mempool] Remove NotifyEntryAdded and NotifyEntryRemoved callbacks.
Jan 11 2021, 13:20
majcosta requested changes to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 12:31
majcosta added a comment to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
In D8867#203607, @PiRK wrote:

I think the clang warning that is treated as an error goes away with the following commit (D8868). Can I just make sure I land them in rapid succession, or do I need to squash them?

Jan 11 2021, 12:25
majcosta added a comment to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 12:23
majcosta added a comment to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.

@bot build-clang

Jan 11 2021, 12:06
majcosta accepted D8866: [validation] Remove conflictedTxs from PerBlockConnectTrace.
Jan 11 2021, 12:05
majcosta accepted D8865: [validation interface] Remove vtxConflicted from BlockConnected.
Jan 11 2021, 11:40
majcosta accepted D8864: [wallet] Notify conflicted transactions in TransactionRemovedFromMempool.
Jan 11 2021, 11:32
majcosta accepted D8863: [FUZZER] Fix build failure.
Jan 11 2021, 11:21
majcosta accepted D8861: [net] Add extra entropy to the version message.
Jan 11 2021, 00:44

Jan 10 2021

majcosta requested changes to D8861: [net] Add extra entropy to the version message.
Jan 10 2021, 08:53
majcosta accepted D8856: [cashtab] Remove autocomplete from address inputs.
Jan 10 2021, 07:19

Jan 9 2021

majcosta accepted D8859: rpc: add missing HelpExampleRpc for getblockfilter.
Jan 9 2021, 05:50
majcosta accepted D8858: Update merkle.cpp.
Jan 9 2021, 05:45
majcosta accepted D8857: Fix UB with bench on genesis block.
Jan 9 2021, 05:13
majcosta requested changes to D8856: [cashtab] Remove autocomplete from address inputs.

can we do that for the seed import textbox too?

Jan 9 2021, 05:10
majcosta accepted D8855: Add bn2vch test to functional tests.
Jan 9 2021, 05:03
majcosta accepted D8852: [tests] remove bignum.py.
Jan 9 2021, 05:02
majcosta closed D8853: [cashtab] update CI to npm7/node15.
Jan 9 2021, 04:41
majcosta committed rABC8c42cddde0b5: [cashtab] update CI to npm7/node15 (authored by majcosta).
[cashtab] update CI to npm7/node15
Jan 9 2021, 04:41

Jan 8 2021

teamcity awarded D8853: [cashtab] update CI to npm7/node15 a Haypence token.
Jan 8 2021, 18:54
majcosta added a comment to D8853: [cashtab] update CI to npm7/node15.

@bot cashtab-tests

Jan 8 2021, 18:54
majcosta requested review of D8853: [cashtab] update CI to npm7/node15.
Jan 8 2021, 18:45
majcosta closed D8849: [cashtab] use npm ci instead of install on CI.
Jan 8 2021, 16:20
majcosta committed rABC937e3797392e: [cashtab] use npm ci instead of install on CI (authored by majcosta).
[cashtab] use npm ci instead of install on CI
Jan 8 2021, 16:20
teamcity awarded D8849: [cashtab] use npm ci instead of install on CI a Mountain of Wealth token.
Jan 8 2021, 15:59
majcosta added a comment to D8849: [cashtab] use npm ci instead of install on CI.

@bot cashtab-tests

Jan 8 2021, 15:58
majcosta updated the diff for D8849: [cashtab] use npm ci instead of install on CI.

rebase

Jan 8 2021, 15:58
teamcity awarded D8849: [cashtab] use npm ci instead of install on CI a Doubloon token.
Jan 8 2021, 15:16
majcosta added a comment to D8849: [cashtab] use npm ci instead of install on CI.

@bot cashtab-tests

Jan 8 2021, 15:16
majcosta updated the diff for D8849: [cashtab] use npm ci instead of install on CI.

fixed backports installation erroring out and fixed the issue with npm breaking when trying to talk to github via ssh

Jan 8 2021, 15:14
majcosta added a comment to D8849: [cashtab] use npm ci instead of install on CI.

Can you add these instructions to the README? (instead of npm install, npm ci)

Jan 8 2021, 15:12
majcosta planned changes to D8849: [cashtab] use npm ci instead of install on CI.

investigating bustedness

Jan 8 2021, 12:50
teamcity awarded D8849: [cashtab] use npm ci instead of install on CI a Piece of Eight token.
Jan 8 2021, 12:31
majcosta added a comment to D8849: [cashtab] use npm ci instead of install on CI.

@bot cashtab-tests

Jan 8 2021, 12:30
majcosta updated the diff for D8849: [cashtab] use npm ci instead of install on CI.

adding nodejs

Jan 8 2021, 12:29
teamcity awarded D8849: [cashtab] use npm ci instead of install on CI a Haypence token.
Jan 8 2021, 12:14
majcosta added a comment to D8849: [cashtab] use npm ci instead of install on CI.

@bot cashtab-tests

Jan 8 2021, 12:13
majcosta requested review of D8849: [cashtab] use npm ci instead of install on CI.
Jan 8 2021, 12:11
majcosta accepted D8844: [CI] Fix gitian script path.
Jan 8 2021, 10:42
majcosta accepted D8839: [CI] Generate coverage reports for cashtab.
Jan 8 2021, 09:17
majcosta accepted D8842: [CI] Improve cashtab reporting in case of a test failure.

looks good

Jan 8 2021, 09:06
majcosta accepted D8770: test: check custom descendant limit in mempool_packages.py.
Jan 8 2021, 00:58
majcosta requested changes to D8839: [CI] Generate coverage reports for cashtab.
Jan 8 2021, 00:35
majcosta added a comment to D8839: [CI] Generate coverage reports for cashtab.

got this when running ./contrib/teamcity/build-configurations.py cashtab-tests

Jan 8 2021, 00:17

Jan 7 2021

majcosta accepted D8837: Implement O(1) OP_IF/NOTIF/ELSE/ENDIF logic.
Jan 7 2021, 18:19
majcosta accepted D8836: [refactor] interpreter: define interface for vfExec.
Jan 7 2021, 18:14
majcosta requested changes to D8836: [refactor] interpreter: define interface for vfExec.

Depends on D8835 right? not D8335

Jan 7 2021, 17:56
majcosta accepted D8835: Benchmark script verification with 100 nested IFs.
Jan 7 2021, 17:28
majcosta accepted D8834: [CASHTAB] Remove the no longer needed --legacy-peer-deps install flag.
Jan 7 2021, 16:53
majcosta accepted D8833: [CASHTAB] Update the README.
Jan 7 2021, 16:53
majcosta requested changes to D8833: [CASHTAB] Update the README.
Jan 7 2021, 16:29
majcosta accepted D8832: [CI] Generate Junit output for the cashtab tests.
Jan 7 2021, 16:09
majcosta accepted D8824: [CI] Run cashtab tests on CI.
Jan 7 2021, 16:04
majcosta accepted D8828: [CASHTAB] Update the jest dependency.
Jan 7 2021, 15:51
majcosta accepted D8827: test: Add bad-txns-*-toolarge test cases to invalid_txs.
Jan 7 2021, 15:25
majcosta accepted D8826: doc: Warn about wallet.dat re-use and backups.
Jan 7 2021, 15:23
majcosta accepted D8821: [CI] Allow for running arbitrary scripts.
Jan 7 2021, 15:21
majcosta accepted D8819: [cashtab] Update css-loader.
Jan 7 2021, 06:04
majcosta accepted D8818: [cashtab] Upgrade mini-css-extract-plugin.
Jan 7 2021, 06:04
majcosta accepted D8814: [cashtab] Update file-loader and fs-extra.
Jan 7 2021, 06:03
majcosta accepted D8813: [cashtab] Upgrade eslint.
Jan 7 2021, 06:03
majcosta accepted D8812: [cashtab] Update webpack.
Jan 7 2021, 06:02
majcosta accepted D8811: [cashtab] Upgrade node modules.
Jan 7 2021, 06:02
majcosta accepted D8816: Do not format autogenerated files.
Jan 7 2021, 05:17

Jan 6 2021

majcosta closed D8804: [backport#13655] bitcoinconsensus: invalid flags error should be set to bitcoinconsensus_err.
Jan 6 2021, 20:51
majcosta committed rABC685164613eda: [backport#13655] bitcoinconsensus: invalid flags error should be set to… (authored by Wladimir J. van der Laan <laanwj@gmail.com>).
[backport#13655] bitcoinconsensus: invalid flags error should be set to…
Jan 6 2021, 20:51
majcosta updated the summary of D8804: [backport#13655] bitcoinconsensus: invalid flags error should be set to bitcoinconsensus_err.
Jan 6 2021, 14:12
majcosta updated the diff for D8804: [backport#13655] bitcoinconsensus: invalid flags error should be set to bitcoinconsensus_err.

readded test case using bitcoinconsensus_SCRIPT_ENABLE_SIGHASH_FORKID instead

Jan 6 2021, 14:11
majcosta accepted D8805: rpc/wallet: initialize nFeeRequired explicitely to 0.
Jan 6 2021, 10:53
majcosta updated the summary of D8804: [backport#13655] bitcoinconsensus: invalid flags error should be set to bitcoinconsensus_err.
Jan 6 2021, 10:52