Skip to content

Conversation

altendky
Copy link
Contributor

Reverts #19804

turns out it's more complicated and this only updated some cases

@altendky altendky requested a review from a team as a code owner July 17, 2025 15:16
@emlowe emlowe added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Jul 17, 2025
@altendky altendky requested a review from emlowe July 17, 2025 16:52
Copy link

Pull Request Test Coverage Report for Build 16349041249

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 48 unchanged lines in 10 files lost coverage.
  • Overall coverage decreased (-0.02%) to 91.347%

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/wallet/util/wallet_sync_utils.py 1 86.07%
chia/daemon/client.py 2 74.16%
chia/timelord/timelord.py 2 80.79%
chia/server/node_discovery.py 3 81.93%
chia/wallet/wallet_node.py 3 87.6%
chia/full_node/full_node.py 12 87.78%
chia/_tests/core/util/test_lockfile.py 22 77.31%
Totals Coverage Status
Change from base Build 16332491766: -0.02%
Covered Lines: 102352
Relevant Lines: 111929

💛 - Coveralls

@Starttoaster Starttoaster merged commit 5da9e79 into main Jul 21, 2025
520 of 522 checks passed
@Starttoaster Starttoaster deleted the revert-19804-zstd_1_5_7_2 branch July 21, 2025 15:54
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 Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants