Remove pruning #42
Open
Remove pruning #42
Conversation
This file contains hidden or 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
- Remove pruning configuration from settings and config files - Update UI to remove pruning-related text and options - Add test script to verify pruning removal - Ensure full node operation is enforced
|
This changes in this PR seem to do more than change pruning options. Is the PR in its expected state? |
I think this change(removing pruning option) does help bitcoin become more decentralized if that's what you're asking. |
|
No - I noticed a number of the changes in the PR are unrelated to the claimed intent. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
### Remove Pruning Option and Related Code
Overview
This PR removes all pruning-related code and configurations, ensuring the application enforces full node operation. This change enhances network health and security by requiring complete blockchain validation.
Changes Made
🗑️ Removed pruning configuration from settings and config files
🖥️ Updated UI to remove pruning-related text and options
✅ Added test script to verify pruning removal
🔒 Ensured full node operation is enforced
Testing
Verified build completes successfully
Confirmed no pruning references remain in source code
Validated settings and config files are clean
Test script passes all checks
Impact
Improves network decentralization
Enhances security through full node validation
Simplifies codebase by removing unused pruning logic
Notes
The test script (
test-pruning-removal.ps1
) has been added to prevent regressions
All existing functionality remains intact, with pruning options removed