Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Oct 3, 2017

Random qa fixups:

  • make cov should work for out-of-tree builds
  • check-rpc-mappings.py should assert that it is actually checking something and the lists are not empty.

@maflcko maflcko added the Tests label Oct 3, 2017
@laanwj
Copy link
Member

laanwj commented Oct 4, 2017

Trivial ACK fae60e3

@laanwj laanwj merged commit fae60e3 into bitcoin:master Oct 4, 2017
laanwj added a commit that referenced this pull request Oct 4, 2017
fae60e3 qa: Fix lcov for out-of-tree builds (MarcoFalke)
fae2673 qa: check-rpc-mapping must not run on empty lists (MarcoFalke)

Pull request description:

  Random qa fixups:

  * `make cov` should work for out-of-tree builds
  * `check-rpc-mappings.py` should assert that it is actually checking something and the lists are not empty.

Tree-SHA512: 2b66f69d6a1ae035c772f8ceb1d58dce904d98058330dad6ccb1421941e167aa748fe1c12126b87f43b0843f51fa85d89de079d586629fcaf8261c44a8dc6053
@maflcko maflcko deleted the Mf1710-qaFixups branch October 4, 2017 12:27
maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Oct 4, 2017
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Feb 9, 2020
…ng check

fae60e3 qa: Fix lcov for out-of-tree builds (MarcoFalke)
fae2673 qa: check-rpc-mapping must not run on empty lists (MarcoFalke)

Pull request description:

  Random qa fixups:

  * `make cov` should work for out-of-tree builds
  * `check-rpc-mappings.py` should assert that it is actually checking something and the lists are not empty.

Tree-SHA512: 2b66f69d6a1ae035c772f8ceb1d58dce904d98058330dad6ccb1421941e167aa748fe1c12126b87f43b0843f51fa85d89de079d586629fcaf8261c44a8dc6053
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants