Back to Codeberg

feat: tag commands #262

forgejo-contrib-forgejo-cli-pulls-262.md

latest11.5 KB
Original Source

forgejo-contrib/forgejo-cli

Watch20

Star402

Fork

You've already forked forgejo-cli

51

CodeIssues 44Pull requests 8Releases 11Packages 2WikiActivity

feat: tag commands #262

Merged

Cyborusmerged 2 commits from tag-commands into main 2026-01-11 16:58:59 +01:00

Conversation 1Commits 2Files changed 2 +132

Cyborus commented 2026-01-10 03:24:54 +01:00

Member

Copy link

Closes #209

Adds:

  • fj tag create <NAME> [--body [BODY]] [--branch <BRANCH>]
  • fj tag delete <NAME>
  • fj tag list [--page <NUM>]
  • fj tag view <NAME>

Closes #209 Adds: - fj tag create \<NAME\> [--body [BODY]] [--branch \<BRANCH\>] - fj tag delete \<NAME\> - fj tag list [--page \<NUM\>] - fj tag view \<NAME\>

Cyborus added 1 commit 2026-01-10 03:24:55 +01:00

feat: tag commands...

All checks were successful

ci/woodpecker/pr/check Pipeline was successful

Details

3d8c3c5cde

Closes[#209](/forgejo-contrib/forgejo-cli/issues/209)Adds:
- `fj tag create <NAME> [--body [BODY]] [--branch <BRANCH>]`
- `fj tag delete <NAME>`
- `fj tag list [--page <NUM>]`
- `fj tag view <NAME>`

Cyborus requested review from LordMZTE 2026-01-10 03:26:59 +01:00

Cyborus added this to the v0.4.0 milestone 2026-01-10 03:28:54 +01:00

LordMZTE requested changes 2026-01-10 18:57:36 +01:00 Dismissed

LordMZTE left a comment

Copy link

Nice! All commands have been tested and work as expected.

Nice! All commands have been tested and work as expected.

src/tag.rs

Show resolved Hide resolved

| | | | @ -0,0 +119,4 @@ | | | | | | let id = tag.id.as_deref().ok_or_eyre("tag does not have name")?; | | | | | | | | | | | | let crate::SpecialRender { bold, reset, .. } = crate::special_render(); | | | | | | println!("{bold}{name}{reset}"); |

LordMZTE commented 2026-01-10 18:56:40 +01:00

Collaborator

Copy link

There seems to be a lot of info missing here. I think all the other fields of the Tag struct we get from the API are potentially interesting, particularly the message, too.

There seems to be a lot of info missing here. I think all the other fields of the Tag struct we get from the API are potentially interesting, particularly the message, too.

Cyborus commented 2026-01-10 19:12:08 +01:00

Author

Member

Copy link

I thought it would be too, but it just seems to be the message of the commit it points to...?

I thought it would be too, but it just seems to be the message of the commit it points to...?

LordMZTE commented 2026-01-10 19:41:04 +01:00

Collaborator

Copy link

Huh, I assumed it would be the tag message. Still, I think we should print out all information we get here as long as it doesn't clutter output, which I think we're very far away from. Thoughts?

Huh, I assumed it would be the tag message. Still, I think we should print out all information we get here as long as it doesn't clutter output, which I think we're very far away from. Thoughts?

Cyborus commented 2026-01-10 20:24:38 +01:00

Author

Member

Copy link

Having done it, I think it looks nicer with it 👍

Having done it, I think it looks nicer with it 👍

Cyborus commented 2026-01-10 20:47:17 +01:00

Author

Member

Copy link

I think the download URLs are simple enough that they don't need included

I think the download URLs are simple enough that they don't need included

LordMZTE marked this conversation as resolved

Cyborus added 1 commit 2026-01-10 20:22:38 +01:00

feat: show commit message in tag view

All checks were successful

ci/woodpecker/pr/check Pipeline was successful

Details

225fa05b51

LordMZTE approved these changes 2026-01-10 21:49:24 +01:00

Cyborus merged commit 0682addfb1 into main 2026-01-11 16:58:59 +01:00

Cyborus deleted branch tag-commands 2026-01-11 16:59:01 +01:00

Cyborus referenced this pull request from a commit 2026-01-11 16:59:01 +01:00 Merge pull request 'feat: tag commands' (#262) from tag-commands into main

Sign in to join this conversation.

Reviewers

No reviewers

LordMZTE

Labels

Clear labels[ Kind/Breaking Breaking change that won't be backward compatible

](#)[ Kind/Bug Something is not working

](#)[ Kind/Design Discussion about UI/UX design

](#)[ Kind/Documentation Documentation changes

](#)[ Kind/Enhancement Improve existing functionality

](#)[ Kind/Feature New functionality

](#)[ Kind/Security This is security issue

](#)[ Kind/Testing Issue or pull request related to testing

](#)[ Kind/Upstream This is an issue with upstream software (Forgejo) that is probably not our fault

](#)

[ Priority

Critical The priority is critical

](#)[ Priority

High The priority is high

](#)[ Priority

Low The priority is low

](#)[ Priority

Medium The priority is medium

](#)

[ Reviewed

Confirmed Issue has been confirmed

](#)[ Reviewed

Duplicate This issue or pull request already exists

](#)[ Reviewed

Invalid Invalid issue

](#)[ Reviewed

Won't Fix This issue won't be fixed

](#)

[ Status

Abandoned Somebody has started to work on this but abandoned work

](#)[ Status

Blocked Something is blocking this issue or pull request

](#)[ Status

Need More Info Feedback is required to reproduce issue or to continue work

](#)

[ Suspicious

](#)

No labels Kind/Breaking Kind/Bug Kind/Design Kind/Documentation Kind/Enhancement Kind/Feature Kind/Security Kind/Testing Kind/Upstream [ Priority

Critical ](/forgejo-contrib/forgejo-cli/pulls?labels=173012) [ Priority

High ](/forgejo-contrib/forgejo-cli/pulls?labels=173013) [ Priority

Low ](/forgejo-contrib/forgejo-cli/pulls?labels=173015) [ Priority

Medium ](/forgejo-contrib/forgejo-cli/pulls?labels=173014) [ Reviewed

Confirmed ](/forgejo-contrib/forgejo-cli/pulls?labels=173007) [ Reviewed

Duplicate ](/forgejo-contrib/forgejo-cli/pulls?labels=173005) [ Reviewed

Invalid ](/forgejo-contrib/forgejo-cli/pulls?labels=173006) [ Reviewed

Won't Fix ](/forgejo-contrib/forgejo-cli/pulls?labels=173008) [ Status

Abandoned ](/forgejo-contrib/forgejo-cli/pulls?labels=173011) [ Status

Blocked ](/forgejo-contrib/forgejo-cli/pulls?labels=173010) [ Status

Need More Info ](/forgejo-contrib/forgejo-cli/pulls?labels=173009) Suspicious

Milestone

Clear milestone

No items

No milestone v0.4.0

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications Subscribe

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

forgejo-contrib/forgejo-cli!262

WritePreview

Loading…

Add table

| Rows | | | Columns | |

CancelOK

Add a link

Url Description Hint: With a URL in your clipboard, you can paste directly into the editor to create a link.

CancelOK

CancelSave

Reference in a new issue

Repository

forgejo-contrib/forgejo-cli

Title

Body

Create issue

No description provided.

Delete branch "tag-commands"

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?

No Yes