chore: disable deposit write commands for v0.1#55
Merged
Conversation
Comment out all deposit subcommands (edit, update, discard, publish) and their corresponding API methods and tests. v0.1 is read-only. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR prepares the v0.1 release as read-only by commenting out all deposit write functionality while preserving the code for future re-enablement. The changes systematically disable deposit subcommands (edit, update, discard, publish) at both the CLI and API layers, leaving only read operations available.
Changes:
- Commented out entire deposit CLI command file including all write subcommands and their registration
- Commented out write API methods (UpdateDeposition, EditDeposition, PublishDeposition, DiscardDeposition) while keeping GetDeposition
- Commented out corresponding write method tests and removed unused
ioimport
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| internal/cli/deposit.go | Entire file wrapped in block comment - disables all deposit CLI commands and subcommands |
| internal/api/depositions.go | Write methods (Update/Edit/Publish/Discard) commented out, GetDeposition remains active |
| internal/api/depositions_test.go | Write method tests commented out, unused io import removed |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
edit,update,discard,publish) and their CLI registrationUpdateDeposition,EditDeposition,PublishDeposition,DiscardDeposition) keeping onlyGetDepositionELI5
v0.1 is a read-only release — you can look at records but not change them. This wraps all the "change stuff" code in comments so it's still there for later but won't run. Will add write to later relaes
Test plan
go build ./...passesgo test ./...passeszenodo.exebuilt locallyzenodo depositis no longer a recognized command🤖 Generated with Claude Code