Skip to content

Commit

Permalink
Remove support for reserve commands. (#339)
Browse files Browse the repository at this point in the history
### Description

Penultimate step of sunsetting the reserve commands.
https://forum.celo.org/t/sunset-of-reserve-commands/8454

### Other changes
 n/a

### Tested
n/a
### Related issues

- Fixes celo-org/celo-blockchain-planning#439

### Backwards compatibility

nope
### Documentation

docs for commands removed

<!-- start pr-codex -->

---

## PR-Codex overview
This PR removes the support for `reserve:*` commands in the CLI as part
of the deprecation process outlined in a forum discussion.

### Detailed summary
- Deleted the following files:
  - `packages/cli/src/commands/reserve/status.ts`
  - `packages/cli/src/commands/reserve/transfergold.ts`
  - `packages/cli/src/commands/reserve/transfergold.test.ts`
- Updated the `changeset/gold-pumas-fry.md` to reflect the major version
change for `@celo/celocli` and document the removal of `reserve:*`
commands.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your
question}`

<!-- end pr-codex -->
  • Loading branch information
aaronmgdr authored Oct 1, 2024
1 parent 416ed00 commit 87223ba
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 197 deletions.
5 changes: 5 additions & 0 deletions .changeset/gold-pumas-fry.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@celo/celocli': major
---

Remove support for reserve:\* commands. As foretold by https://forum.celo.org/t/sunset-of-reserve-commands/8454
28 changes: 0 additions & 28 deletions packages/cli/src/commands/reserve/status.ts

This file was deleted.

107 changes: 0 additions & 107 deletions packages/cli/src/commands/reserve/transfergold.test.ts

This file was deleted.

62 changes: 0 additions & 62 deletions packages/cli/src/commands/reserve/transfergold.ts

This file was deleted.

0 comments on commit 87223ba

Please sign in to comment.