Skip to content

[CHIA-3604] Port easy CAT endpoints to @marshal #19992

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 21 commits into from
Sep 23, 2025
Merged

[CHIA-3604] Port easy CAT endpoints to @marshal #19992

merged 21 commits into from
Sep 23, 2025
+347 −205

Conversation

Quexington
Copy link
Contributor

Another PR following #18593

@Quexington Quexington requested a review from a team as a code owner August 26, 2025 18:11
@Quexington Quexington added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Aug 26, 2025
aqk
aqk previously approved these changes Aug 26, 2025
View reviewed changes
@Quexington Quexington force-pushed the quex.port_easy_cat_endpoints_to_marshal branch from 89b6919 to 1426956 Compare September 15, 2025 21:22
@Quexington Quexington force-pushed the quex.port_easy_cat_endpoints_to_marshal branch from 1426956 to 95a22d9 Compare September 15, 2025 21:22
Base automatically changed from quex.port_send_notifications_to_streamable to main September 19, 2025 20:27
@pmaslana pmaslana dismissed aqk’s stale review September 19, 2025 20:27

The base branch was changed.

@Quexington Quexington closed this Sep 22, 2025
@Quexington Quexington reopened this Sep 22, 2025
@coveralls-official Coveralls Official
Copy link

Pull Request Test Coverage Report for Build 17920850181

Details

  • 189 of 189 (100.0%) changed or added relevant lines in 10 files are covered.
  • 245 unchanged lines in 15 files lost coverage.
  • Overall coverage increased (+0.008%) to 91.176%
Files with Coverage Reduction New Missed Lines %
chia/daemon/client.py 1 74.16%
chia/daemon/keychain_proxy.py 1 70.57%
chia/full_node/full_node.py 1 87.27%
chia/server/node_discovery.py 1 83.01%
chia/timelord/timelord_api.py 1 93.02%
chia/wallet/util/wallet_sync_utils.py 1 85.57%
chia/_tests/wallet/rpc/test_wallet_rpc.py 2 99.02%
chia/timelord/timelord_launcher.py 2 90.71%
chia/wallet/wallet_request_types.py 2 99.35%
chia/farmer/farmer.py 3 73.0%
Totals Coverage Status
Change from base Build 17869075229: 0.008%
Covered Lines: 102802
Relevant Lines: 112589
💛 - Coveralls

@Quexington Quexington added the ready_to_merge Submitter and reviewers think this is ready label Sep 23, 2025
@pmaslana pmaslana merged commit b45f677 into main Sep 23, 2025
895 of 899 checks passed
@pmaslana pmaslana deleted the quex.port_easy_cat_endpoints_to_marshal branch September 23, 2025 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@altendky altendky altendky approved these changes

@markelrod markelrod markelrod approved these changes

@aqk aqk aqk left review comments

Assignees
No one assigned
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Quexington @aqk @altendky @markelrod @pmaslana