forgejo-contrib-forgejo-cli-pulls-533.md
Watch20
Star402
Fork
You've already forked forgejo-cli
CodeIssues 44Pull requests 8Releases 11Packages 2WikiActivity
Merged
Cyborusmerged 1 commit from Fluffinity/forgejo-cli:feat-repo-watch into main 2026-07-06 22:48:49 +02:00
Conversation 6Commits 1Files changed 2 +134 -1
Fluffinity commented 2026-06-27 14:04:08 +02:00
Contributor
Copy link
This PR adds the commands repo watch, repo unwatch and repo watch-status. The first two commands also check whether the current user is already (un-)subscribed to the corresponding repo. repo watch-status can also be used to list all currently watched repos. Note, that the status check currently uses a 404 as "is not subscribed to the repo", because that is the behavior from codeberg.org. This feels like a hack, but oh well it is what it is.
repo watch, repo unwatch and repo watch-status. The first two commands also check whether the current user is already (un-)subscribed to the corresponding repo. repo watch-status can also be used to list all currently watched repos. Note, that the status check currently uses a 404 as "is not subscribed to the repo", because that is the behavior from codeberg.org. This feels like a hack, but oh well it is what it is. ### Code of Conduct - [x] I agree to act in accordance with the CoC & AI Agreement. - [x] This contribution was not generated by an LLM, even in part.👍 1
Fluffinity added 1 commit 2026-06-27 14:04:10 +02:00
feat(repo): added watch, unwatch and watch-status commands...
Some checks failed
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
ci/woodpecker/pr/check Pipeline failed
These commands allow the user to watch and unwatch repos, as well as
checking in on whether they currently watch a repo. Users can
additionally view all their watched repos by using the --list flag for
the watch-status command.
Fluffinity commented 2026-06-27 14:05:44 +02:00
Author
Contributor
Copy link
This implements #529
This implements #529
Fluffinity force-pushed feat-repo-watch from af355d06e5
Some checks failed
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
ci/woodpecker/pr/check Pipeline failed
to 61aa26489a
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
2026-06-27 14:07:47 +02:00 Compare
Fluffinity force-pushed feat-repo-watch from 61aa26489a
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
to 223ac97500
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-06-27 14:13:20 +02:00 Compare
Fluffinity force-pushed feat-repo-watch from 223ac97500
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
to c5c201ecbd
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-06-27 20:08:01 +02:00 Compare
jpk68 reviewed 2026-06-29 03:57:05 +02:00
localization/en-US/messages.ftl Outdated
Show resolved Hide resolved
| | | | @ -1395,1 +1395,3 @@ |
| | | | | |
| | | | | msg-repo-watch-success = You are now watching {$owner}/{$repo} |
| | | | | msg-repo-watch-already_subscribed = You already watch this repo |
| | | | | msg-repo-unwatch-already_unsubscribed = You already don't watch this repo |
jpk68 commented 2026-06-29 03:57:05 +02:00
Contributor
Copy link
Nit: it might be better to say something like "you don't yet watch this repo"
Nit: it might be better to say something like "you don't yet watch this repo"
Cyborus commented 2026-06-29 05:54:35 +02:00
Member
Copy link
I prefer the current phrasing.
"you don't yet [..]" implies to me that you will watch it in the future, which seems like a strange thing to suggest when trying to _un_watch it.
I prefer the current phrasing. "you don't yet [..]" implies to me that you will watch it in the future, which seems like a strange thing to suggest when trying to *un*watch it.
Fluffinity commented 2026-06-29 13:48:03 +02:00
Author
Contributor
Copy link
I am with Cyborus on this one. "You don't yet watch this repo" sounds weird to me in this context
I am with Cyborus on this one. "You don't yet watch this repo" sounds weird to me in this context
jpk68 commented 2026-06-29 16:16:44 +02:00
Contributor
Copy link
Makes sense. Thanks anyways :)
Makes sense. Thanks anyways :)
Fluffinity marked this conversation as resolved
Fluffinity force-pushed feat-repo-watch from c5c201ecbd
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
to a92afb5c9f
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-06-29 13:46:06 +02:00 Compare
Fluffinity commented 2026-06-29 13:48:18 +02:00
Author
Contributor
Copy link
Rebased onto current main
Rebased onto current main
Cyborus reviewed 2026-07-06 21:23:06 +02:00
src/repo.rs Outdated
Show resolved Hide resolved
| | | | @ -437,0 +455,4 @@ |
| | | | | #[clap(long, short = 'R')] |
| | | | | remote: Option<String>, |
| | | | | #[clap(long)] |
| | | | | list: bool, |
Cyborus commented 2026-07-06 21:23:06 +02:00
Member
Copy link
I think --list should conflicts_with the repo arg.
I think --list should conflicts_with the repo arg.
Fluffinity commented 2026-07-06 22:25:31 +02:00
Author
Contributor
Copy link
Makes sense. I updated this
Makes sense. I updated this
Fluffinity marked this conversation as resolved
Fluffinity force-pushed feat-repo-watch from a92afb5c9f
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
to 1ba6d8ad94
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-07-06 22:25:22 +02:00 Compare
Fluffinity force-pushed feat-repo-watch from 1ba6d8ad94
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
to f586c87dcd
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-07-06 22:32:50 +02:00 Compare
Cyborus approved these changes 2026-07-06 22:36:04 +02:00 Dismissed
Fluffinity force-pushed feat-repo-watch from f586c87dcd
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
to b5d9838f41
All checks were successful
ci/woodpecker/pr/check-typos Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/check-clippy Pipeline was successful
2026-07-06 22:37:06 +02:00 Compare
Fluffinity dismissed Cyborus's review 2026-07-06 22:37:06 +02:00
New commits pushed, approval review dismissed automatically according to repository settings
Fluffinity commented 2026-07-06 22:37:40 +02:00
Author
Contributor
Copy link
This was just a rebase onto current main
This was just a rebase onto current main
Cyborus approved these changes 2026-07-06 22:47:50 +02:00
Cyborus referenced this pull request from a commit 2026-07-06 22:48:49 +02:00 Merge pull request 'feat(repo): add watch, unwatch and watch-status commands' (#533) from Fluffinity/forgejo-cli:feat-repo-watch into main
Cyborus merged commit 6b338d8372 into main 2026-07-06 22:49:05 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
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
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
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!533
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 "Fluffinity/forgejo-cli:feat-repo-watch"
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