-
Notifications
You must be signed in to change notification settings - Fork 2k
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
catchup: long_lived/vault from main @ b4137651b6b2107f88b4a83e64b7b2c918c5c3f3 #18652
Open
Quexington
wants to merge
26
commits into
long_lived/vault
Choose a base branch
from
catchup/long_lived_vault_from_main_b4137651b6b2107f88b4a83e64b7b2c918c5c3f3
base: long_lived/vault
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
catchup: long_lived/vault from main @ b4137651b6b2107f88b4a83e64b7b2c918c5c3f3 #18652
Quexington
wants to merge
26
commits into
long_lived/vault
from
catchup/long_lived_vault_from_main_b4137651b6b2107f88b4a83e64b7b2c918c5c3f3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* slightly simplify call to pre_validate_blocks_multiprocessing. batch size is always 4, check_filter is always True. Remove the member function wrapper around it in Blockchain, to prepare for being able to pass wrapped blockchains into it * address review comments * Apply suggestions from code review (addendum). --------- Co-authored-by: Amine Khaldi <[email protected]>
… cost limit (#18574) Deduct block overhead from the mempool's maximum block clvm cost limit.
Bumps [setuptools](https://github.com/pypa/setuptools) from 71.1.0 to 75.1.0. - [Release notes](https://github.com/pypa/setuptools/releases) - [Changelog](https://github.com/pypa/setuptools/blob/main/NEWS.rst) - [Commits](pypa/setuptools@v71.1.0...v75.1.0) --- updated-dependencies: - dependency-name: setuptools dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…_coins (#18622) Extend coins without building lists on each step in combine_coins.
…8621) Remove no longer needed TestWalletInterestedStore.
Annotate test_wallet_key_val_store.py.
…ault cats in test_cat_endpoints (#18587) Check the cat list values from get_cat_list against the default cats in test_cat_endpoints.
* Port `log_in` to decorator * Port `get_logged_in_fingerprint` to decorator * Port `get_public_keys` to decorator * Port `get_private_key` to decorator * Port `get_mnemonic` to decorator * Port `add_key` to decorator * Port `delete_key` to decorator * Port `check_delete_key` to decorator * Port `delete_all_keys` to decorator * Bad copypasta
…8601) * Tweak `TransactionEndpointRequest` to require TX args * whoops * Add an option around the ban
Update to macOS 13 for build and test
…8628) Simplify double spend validation in validate_block_body.
* bump chia_rs to 0.14.0 * update tests covering negative division (allowed since the hard fork) and infinity G1 keys (disallowed with soft fork 5) * BLSCache.update() takes GTElement now * use get_conditions_from_spendbundle() in test_mempool_manager, to get the ANALYZE_SPENDS behavior * fix mypy warnings
adding ca updates Co-authored-by: ChiaAutomation <[email protected]>
* minor simplification of stacked if-conditions and early exits on failure paths * Simplify NPCResult -> SpendBundleConditions * make include_spends() take SpendBundleConditions, rather than NPCResult
Updated GUI pin
* Reduce level log spam when downloading files * black fix * Just change info->debug
Annotate test_wallet_interested_store.py.
…s spend bundle confusion (#18639) Simplify calling pre_validate_spendbundle and avoid spend vs spend bundle confusion.
Bumps [chiavdf](https://github.com/Chia-Network/chiavdf) from 1.1.4 to 1.1.6. - [Release notes](https://github.com/Chia-Network/chiavdf/releases) - [Commits](Chia-Network/chiavdf@1.1.4...1.1.6) --- updated-dependencies: - dependency-name: chiavdf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…nerations (#18612) * Some logging and some code to limit full file generation based on the max number of full files allowed * Don't write out full files that aren't needed * Black fixes * Adjust the full file during error conditions * No need for try
maybe fixup `main` coverage diff
…long_lived_vault_from_main_b4137651b6b2107f88b4a83e64b7b2c918c5c3f3
Removed dependencies detected. Learn more about Socket for GitHub ↗︎ 🚮 Removed packages: pypi/[email protected], pypi/[email protected], pypi/[email protected] |
Quexington
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
Sep 30, 2024
|
Pull Request Test Coverage Report for Build 11114460679Details
💛 - 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
Exclude_Notes
Use this label if the changes in the PR should be excluded from the release notes
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Source hash: b413765
Remaining commits: 0
Merge diffs: https://quexington.github.io/merge-diffs/e1d477943ee68ec5d74ef880c8a73a0309d5fe8c.html