Aviac/codeberg-cli
10
295
Fork 33
Code Issues 13 Pull requests 2 Projects Releases 6 Packages Wiki Activity

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
Conversation 7 Commits 2 Files changed 4 +269
timptner commented 2025-06-18 00:17:53 +02:00
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.
DrDaveD commented 2025-09-05 00:32:05 +02:00
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.
timptner commented 2025-09-09 16:29:29 +02:00
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.
DrDaveD commented 2025-09-09 16:55:30 +02:00
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).
Aviac commented 2025-09-10 08:59:19 +02:00
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!
❤️ 1
Aviac commented 2025-09-10 09:02:14 +02:00
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.
Aviac force-pushed feat/release from e877ec6768 to 29c2cc93a2 2025-09-15 20:05:32 +02:00 Compare
Aviac commented 2025-09-15 20:20:18 +02:00
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
timptner commented 2025-09-17 09:45:59 +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!
❤️ 1

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
actions

   
bug

Something is not working

  
build

   
chore

   
CI

   
cleanup

   
CLI

   
contribution welcome

Contributions are very welcome, get started here

  
docs

No description

  
duplicate

This issue or pull request already exists

  
enhancement

   
feature

   
fix

   
good first issue

Interested in contributing? Get started here.

  
help wanted

Need some help

  
invalid

Something is wrong

  
perf

   
question

More information is needed

  
refactor

   
release

   
revert

   
style

   
test

   
tracking

No description

  
UI

   
upstream

Related to an upstream repository, already reported there

  
UX
No labels
actions
bug
build
chore
CI
cleanup
CLI
contribution welcome
docs
duplicate
enhancement
feature
fix
good first issue
help wanted
invalid
perf
question
refactor
release
revert
style
test
tracking
UI
upstream
UX
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
No description provided.