Skip to content

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Jul 9, 2025

Purpose:

the wallet was not updating constant overrides correctly as some constants have been renamed

@arvidn arvidn requested a review from a team as a code owner July 9, 2025 18:17
@arvidn arvidn added Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Jul 9, 2025
@arvidn arvidn requested review from emlowe and wjblanke July 9, 2025 18:18
Copy link

Pull Request Test Coverage Report for Build 16177066668

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • 27 unchanged lines in 11 files lost coverage.
  • Overall coverage increased (+0.004%) to 91.392%

Files with Coverage Reduction New Missed Lines %
chia/daemon/keychain_proxy.py 1 73.24%
chia/farmer/farmer.py 1 73.28%
chia/full_node/pending_tx_cache.py 1 96.55%
chia/rpc/rpc_server.py 1 88.24%
chia/wallet/util/wallet_sync_utils.py 1 86.07%
chia/daemon/client.py 2 74.16%
chia/server/node_discovery.py 2 82.29%
chia/_tests/core/util/test_lockfile.py 3 88.89%
chia/wallet/wallet_node.py 3 87.53%
chia/server/address_manager.py 5 92.84%
Totals Coverage Status
Change from base Build 16149582872: 0.004%
Covered Lines: 102073
Relevant Lines: 111567

💛 - Coveralls

@cmmarslender cmmarslender merged commit 37d0a3c into main Jul 10, 2025
520 of 522 checks passed
@cmmarslender cmmarslender deleted the update-testnet-overrides-wallet branch July 10, 2025 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants