Skip to content
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.

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

Add @tx_out_cmd decorator #17842

Merged
merged 2 commits into from
May 9, 2024
Merged

Conversation

Quexington
Copy link
Contributor

This adds a new command decorator (unused for now but more PRs to come) that automatically adds the functionality to export transactions rather than automatically push them.

@Quexington Quexington added Added Required label for PR that categorizes merge commit message as "Added" for changelog vault labels Apr 8, 2024
@Quexington Quexington marked this pull request as ready for review April 8, 2024 19:57
@Quexington Quexington requested a review from a team as a code owner April 8, 2024 19:57
…c224fb5744313872b5b6' into quex.add_tx_out_decorator
@Quexington Quexington changed the base branch from long_lived/vault to catchup/long_lived_vault_from_main_02a7e8b1b55b0f43add5c224fb5744313872b5b6 April 9, 2024 14:35
Base automatically changed from catchup/long_lived_vault_from_main_02a7e8b1b55b0f43add5c224fb5744313872b5b6 to long_lived/vault April 16, 2024 14:13
@Quexington Quexington closed this Apr 16, 2024
@Quexington Quexington reopened this Apr 16, 2024
@Quexington Quexington closed this Apr 25, 2024
@Quexington Quexington reopened this Apr 25, 2024
@Quexington Quexington closed this May 6, 2024
@Quexington Quexington reopened this May 6, 2024
@Quexington Quexington closed this May 7, 2024
@Quexington Quexington reopened this May 7, 2024
Copy link

Pull Request Test Coverage Report for Build 8989596702

Details

  • 33 of 33 (100.0%) changed or added relevant lines in 2 files are covered.
  • 8 unchanged lines in 5 files lost coverage.
  • Overall coverage increased (+0.01%) to 90.842%

Files with Coverage Reduction New Missed Lines %
chia/wallet/wallet_node.py 1 88.97%
chia/server/node_discovery.py 1 79.96%
chia/daemon/server.py 1 83.22%
chia/farmer/farmer_api.py 2 94.4%
chia/rpc/rpc_server.py 3 87.67%
Totals Coverage Status
Change from base Build 8989592579: 0.01%
Covered Lines: 99537
Relevant Lines: 109512

💛 - Coveralls

@Quexington Quexington requested a review from wjblanke May 7, 2024 19:39
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aok

@Quexington Quexington merged commit d352c11 into long_lived/vault May 9, 2024
1548 of 1559 checks passed
@Quexington Quexington deleted the quex.add_tx_out_decorator branch May 9, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog vault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants