Skip to content

feat: add delete subcommand to remove wallet config file - #316

Open
ayush-codes6439 wants to merge 1 commit into
bitcoindevkit:masterfrom
ayush-codes6439:feature/delete-wallet-config
Open

feat: add delete subcommand to remove wallet config file#316
ayush-codes6439 wants to merge 1 commit into
bitcoindevkit:masterfrom
ayush-codes6439:feature/delete-wallet-config

Conversation

@ayush-codes6439

Copy link
Copy Markdown

Description

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature
  • I've updated CHANGELOG.md

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

Description

Adds a new subcommand to delete the stored wallet configuration file from disk.

Notes to the reviewers

This PR introduces the functionality to safely locate and remove the wallet config file when the delete command is executed.

Changelog notice

  • Added delete-wallet-config command to remove existing wallet configuration.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature
  • I've updated CHANGELOG.md

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

@ayush-codes6439

Copy link
Copy Markdown
Author

Hi maintainers,

I have opened this PR to add the subcommand for deleting the wallet configuration file.

Summary of changes:

  • Added functionality to safely delete the wallet config file.
  • Ran cargo fmt and cargo clippy locally.

Please review the changes when you get a chance and let me know if any updates are needed. Thanks!

@ayush-codes6439
ayush-codes6439 force-pushed the feature/delete-wallet-config branch from e2b9404 to 6a852be Compare August 20, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant