Add stackit git commands: list, describe, create and delete.#725
Merged
bahkauv70 merged 8 commits intostackitcloud:mainfrom May 13, 2025
Merged
Add stackit git commands: list, describe, create and delete.#725bahkauv70 merged 8 commits intostackitcloud:mainfrom
bahkauv70 merged 8 commits intostackitcloud:mainfrom
Conversation
* Add stackit git list instances command * Add stackit git list instances command * Git. Move list package and add describe command * Stackit Git create instance command * Add delete command to the stackit git cli
peremanent
requested review from
Fyusel,
bahkauv70,
marceljk and
rubenhoenle
as code owners
rubenhoenle
reviewed
May 5, 2025
rubenhoenle
reviewed
May 5, 2025
rubenhoenle
reviewed
May 5, 2025
rubenhoenle
reviewed
May 5, 2025
Member
|
@peremanent FYI: The CI run failed because docs aren't up to-to-date. Please re-generate the docs with |
…eneration * Stop using new variables for the globalflags.ProjectIdFlag and minor issues * Add docs of the stackit git instance command
bahkauv70
suggested changes
May 8, 2025
Contributor
bahkauv70
left a comment
bahkauv70
left a comment
There was a problem hiding this comment.
Added some minor stuff. When this is fixed, the PR may be merged
… *params.CmdParams attribute
… *params.CmdParams attribute Merge with upstream and modify stackit git commands to use new params *params.CmdParams attribute
bahkauv70
suggested changes
May 13, 2025
Contributor
bahkauv70
left a comment
bahkauv70
left a comment
There was a problem hiding this comment.
Looks good now, only the pipeline fails. The reason could be, that snyk is not able to handle the toolchain option from go.mod. Could you please remove this and try again? If the pipeline is green, we can merge this
Contributor
|
synk still has an issue, we'll check this again |
bahkauv70
approved these changes
May 13, 2025
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
3 participants
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.
Description
Checklist
make fmtmake generate-docs(will be checked by CI)make test(will be checked by CI)make lint(will be checked by CI)