Skip to content

Conversation

@Quexington
Copy link
Contributor

The purpose of this PR is to extract some peer discovery logic from the RPC. There was a related change made to make WSM's get_dl_wallet method optionally create the new DL wallet.

@Quexington Quexington requested a review from a team as a code owner December 5, 2025 15:39
@Quexington Quexington changed the title Extract track_new_launcher_id from dl_track_new [LABS-311] Extract track_new_launcher_id from dl_track_new Dec 5, 2025
@Quexington Quexington added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Dec 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

File Coverage Missing Lines
chia/data_layer/data_layer_wallet.py 92.3% lines 889
Total Missing Coverage
33 lines Unknown 96%

@coveralls-official
Copy link

Pull Request Test Coverage Report for Build 19967961008

Details

  • 32 of 33 (96.97%) changed or added relevant lines in 3 files are covered.
  • 32 unchanged lines in 6 files lost coverage.
  • Overall coverage decreased (-0.02%) to 90.755%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chia/data_layer/data_layer_wallet.py 12 13 92.31%
Files with Coverage Reduction New Missed Lines %
chia/timelord/timelord_launcher.py 2 70.0%
chia/full_node/full_node.py 3 87.0%
chia/server/server.py 3 85.18%
chia/wallet/wallet_node.py 6 86.25%
chia/daemon/server.py 7 80.62%
chia/timelord/timelord.py 11 72.0%
Totals Coverage Status
Change from base Build 19938390251: -0.02%
Covered Lines: 102449
Relevant Lines: 112712

💛 - Coveralls

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 coverage-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants