9
311
Fork
You've already forked codeberg-cli
33

feat: add commands for release actions #223

Closed
timptner wants to merge 2 commits from timptner/codeberg-cli:feat/release into main
pull from: timptner/codeberg-cli:feat/release
merge into: Aviac:main
Aviac:main
First-time contributor
Copy link

It's my first pull-request, so be kind with me. 🙂

For my own CI/CD pipelines I want to be able to create releases. Your codeberg CLI project seems really fancy, but it's missing commands to interact with releases. (see #17)

I tried my best to mimic your code style and only created the release list and release create subcommands for now. I'm open for feedback and/or tips to improve my PR. If you're fine with my commits I'm also willing to implement some more release subcommands and extend the list of optional paramaters.

It's my first pull-request, so be kind with me. 🙂 For my own CI/CD pipelines I want to be able to create releases. Your codeberg CLI project seems really fancy, but it's missing commands to interact with releases. (see #17) I tried my best to mimic your code style and only created the `release list` and `release create` subcommands for now. I'm open for feedback and/or tips to improve my PR. If you're fine with my commits I'm also willing to implement some more `release` subcommands and extend the list of optional paramaters.
First-time contributor
Copy link

Any response from the maintainers?

I have a need for a release upload command.

Any response from the maintainers? I have a need for a release upload command.
Author
First-time contributor
Copy link

Sadly, no. But you can check out my fork and then use the two commands.

Sadly, no. But you can check out my fork and then use the two commands.
First-time contributor
Copy link

For now I'm just directly using the API.

For now I'm just [directly using the API](https://codeberg.org/cvmfs-contrib/config-repo/src/branch/master/geoipdb/codeberg-release-upload).
Owner
Copy link

Hi, I'm here. I also partially reviewed this yesterday evening. I will probably merge soon as this looks pretty nice. Thanks for the contribution btw and sorry to let this sit for so long!

Hi, I'm here. I also partially reviewed this yesterday evening. I will probably merge soon as this looks pretty nice. Thanks for the contribution btw and sorry to let this sit for so long!
Owner
Copy link

If you want to help me out a little more i may have a little request:

  • Could you come up with a sequence of CLI commands to test your feature from the user side please?

I'm planning on extending my integration testing to make contributions and reviews easier.

If you want to help me out a little more i may have a little request: - Could you come up with a sequence of CLI commands to test your feature from the user side please? I'm planning on extending my integration testing to make contributions and reviews easier.
Owner
Copy link

Superseded by #234 which includes your changes! Thanks a lot for the contribution, I would've merged it as it was but there were some changes in main that I needed to include here and I didn't want to bother you with it. Hope that's fine with you.

Once again, thanks! 💚

Superseded by #234 which includes your changes! Thanks a lot for the contribution, I would've merged it as it was but there were some changes in main that I needed to include here and I didn't want to bother you with it. Hope that's fine with you. Once again, thanks! 💚
Aviac closed this pull request 2025年09月15日 20:20:18 +02:00
Author
First-time contributor
Copy link

Currently on vacation - sorry for my late response.

It's great to see my commits included. Thank you for the update!

Currently on vacation - sorry for my late response. It's great to see my commits included. Thank you for the update!

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Aviac/codeberg-cli!223
Reference in a new issue
Aviac/codeberg-cli
No description provided.
Delete branch "timptner/codeberg-cli:feat/release"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?