[Nitro CLI] Add stage delete command#9417
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a new nitro stage delete command backed by a new forceDeleteStageById GraphQL mutation, and updates the CLI GraphQL schema/operation documents accordingly.
Changes:
- Add a new
stage deletesubcommand (DeleteStageCommand) that callsforceDeleteStageById. - Add the GraphQL mutation document used by the command.
- Update
schema.graphqlto includeforceDeleteStageById(and a large set of additional schema directive/annotation updates).
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
src/Nitro/CommandLine/src/CommandLine/schema.graphql |
Adds forceDeleteStageById input/payload/error union and updates many fields with directives (@cost, @listSize, @authorize). |
src/Nitro/CommandLine/src/CommandLine/Commands/Stages/StageCommand.cs |
Registers the new delete subcommand under stage. |
src/Nitro/CommandLine/src/CommandLine/Commands/Stages/DeleteStageCommand.graphql |
Defines the mutation operation used by the CLI to force-delete a stage and return the updated stages list. |
src/Nitro/CommandLine/src/CommandLine/Commands/Stages/DeleteStageCommand.cs |
Implements the nitro stage delete command, including confirmation and result formatting. |
Comments suppressed due to low confidence (1)
src/Nitro/CommandLine/src/CommandLine/schema.graphql:3483
- The deprecation reason string has a grammar issue: "Not longer in use" should be "No longer in use".
throughput(from: DateTime! operationKinds: [OperationKind!] @deprecated(reason: "Not longer in use") to: DateTime!): OperationThroughputGraph
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tobias-tengler
left a comment
There was a problem hiding this comment.
You have to re-generate the persisted operations the command for it is in the README.md in the CLI project
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #9417 +/- ##
============================
============================
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.