Skip to content

chore: remove some unused files#296

Merged
xdustinface merged 1 commit intov0.42-devfrom
chore/unused-files
Feb 27, 2026
Merged

chore: remove some unused files#296
xdustinface merged 1 commit intov0.42-devfrom
chore/unused-files

Conversation

@xdustinface
Copy link
Collaborator

Just removing some file clutter.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 20, 2025

Warning

Rate limit exceeded

@xdustinface has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 12 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 26e7bc4 and 6740bab.

📒 Files selected for processing (16)
  • .actrc
  • TEST_SUMMARY.md
  • TODOS.md
  • dash-spv/CODE_ANALYSIS_SUMMARY.md
  • dash/contrib/act/event.json
  • docs/implementation-notes/BLOOM_FILTER_SPEC.md
  • docs/implementation-notes/CHAINLOCK_IMPLEMENTATION.md
  • docs/implementation-notes/CHECKPOINT_IMPLEMENTATION.md
  • docs/implementation-notes/IMPLEMENTATION_STATUS.md
  • docs/implementation-notes/MEMPOOL_IMPLEMENTATION_SUMMARY.md
  • docs/implementation-notes/PEER_REPUTATION_SYSTEM.md
  • docs/implementation-notes/WALLET_SPV_INTEGRATION.md
  • key-wallet-ffi/include/key_wallet_ffi_test.h
  • key-wallet-manager/SPV_WALLET_GUIDE.md
  • test_checksum.rs
  • test_smart_algo.sh
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/unused-files

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@QuantumExplorer
Copy link
Member

I don't think we really want to delete everything here.

Copy link
Member

@QuantumExplorer QuantumExplorer left a comment

Choose a reason for hiding this comment

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

I would keep AGENTS.md, ARCHITECTURE.md, SPV_WALLET_GUIDE.md at the minimum, probably some others as well. (they might need to be regenerated/fixed)

@github-actions github-actions bot added the merge-conflict The PR conflicts with the target branch. label Dec 22, 2025
@github-actions
Copy link

This PR has merge conflicts with the base branch. Please rebase or merge the base branch into your branch to resolve them.

@xdustinface xdustinface marked this pull request as draft December 22, 2025 23:55
@xdustinface xdustinface changed the title chore: Remove some unused files chore: remove some unused files Dec 29, 2025
@xdustinface xdustinface changed the base branch from v0.41-dev to v0.42-dev December 30, 2025 22:35
@xdustinface xdustinface marked this pull request as ready for review February 25, 2026 02:40
@xdustinface xdustinface requested a review from ZocoLini February 25, 2026 02:41
@github-actions github-actions bot removed the merge-conflict The PR conflicts with the target branch. label Feb 25, 2026
Copy link
Collaborator

Choose a reason for hiding this comment

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

Doesnt the .h files regenerate with the current configuration??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not this one with _test postfix. It's some leftover.

@xdustinface xdustinface merged commit 964c137 into v0.42-dev Feb 27, 2026
53 checks passed
@xdustinface xdustinface deleted the chore/unused-files branch February 27, 2026 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants