Skip to content

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Jun 27, 2025

Another PR following #18593

@Quexington Quexington requested a review from a team as a code owner June 27, 2025 14:36
@Quexington Quexington added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Jun 27, 2025
Copy link

coveralls-official bot commented Jun 27, 2025

Pull Request Test Coverage Report for Build 16315677956

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 100 of 103 (97.09%) changed or added relevant lines in 10 files are covered.
  • 462 unchanged lines in 41 files lost coverage.
  • Overall coverage decreased (-0.06%) to 91.332%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chia/cmds/wallet_funcs.py 3 6 50.0%
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/_tests/core/mempool/test_mempool_manager.py 1 99.93%
chia/_tests/wallet/cat_wallet/test_trades.py 1 99.8%
chia/wallet/cat_wallet/cat_outer_puzzle.py 1 98.81%
chia/wallet/nft_wallet/metadata_outer_puzzle.py 1 97.14%
chia/wallet/nft_wallet/singleton_outer_puzzle.py 1 98.77%
chia/daemon/client.py 2 74.16%
Totals Coverage Status
Change from base Build 16149582872: -0.06%
Covered Lines: 102337
Relevant Lines: 111929

💛 - Coveralls

@Quexington Quexington changed the title Port get_wallets to @marshal decorator [CHIA-3293] Port get_wallets to @marshal decorator Jul 1, 2025
@Quexington Quexington closed this Jul 8, 2025
@Quexington Quexington reopened this Jul 8, 2025
@Quexington Quexington requested a review from altendky July 15, 2025 07:15
Copy link
Contributor

File Coverage Missing Lines
chia/wallet/wallet_rpc_api.py 94.1% lines 1035
Total Missing Coverage
120 lines Unknown 99%

@Starttoaster Starttoaster merged commit 018c3fe into main Jul 16, 2025
523 of 526 checks passed
@Starttoaster Starttoaster deleted the quex.port_get_wallet_to_marshal branch July 16, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants