JOJ/Joint-Teapot
13
3
Fork 2
Code Issues 2 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat: new commands in mm channels and gitea issues #6

Open
蔡雨翔524370910013 wants to merge 12 commits from arthurcai/Joint-Teapot:master into master
pull from: arthurcai/Joint-Teapot:master
merge into: JOJ:master
Conversation 3 Commits 12 Files Changed 6 +793 -18
蔡雨翔524370910013 commented 2025-09-19 19:50:28 +08:00
First-time contributor
Copy Link

Main Changes

Doc

  • Updated README and added README for new commands

Gitea Issues

  • Added command to close issues
  • Added command to delete issue labels
  • Added command to label issues (if not exist, will create new labels)

Mattermost

  • Added command to update mm channels (group and personal)
## Main Changes ### Doc - Updated README and added README for new commands ### Gitea Issues - Added command to close issues - Added command to delete issue labels - Added command to label issues (if not exist, will create new labels) ### Mattermost - Added command to update mm channels (group and personal)
蔡雨翔524370910013 added 7 commits 2025-09-19 19:50:29 +08:00
蔡雨翔524370910013 commented 2025-09-19 19:52:37 +08:00
Author
First-time contributor
Copy Link

The fixes the problem of #3. Not sure whether code quality/effects is okay... The label adding and deleting is tested in engr151/test-repo.

The fixes the problem of #3. Not sure whether code quality/effects is okay... The label adding and deleting is tested in engr151/test-repo.
蔡雨翔524370910013 added 1 commit 2025-09-19 22:54:45 +08:00
蔡雨翔524370910013 added 2 commits 2025-09-20 14:29:18 +08:00
蔡雨翔524370910013 commented 2025-09-20 14:30:02 +08:00
Author
First-time contributor
Copy Link

pre-commit check passed after switching to pycln 2.5.0

pre-commit check passed after switching to pycln 2.5.0
张泊明518370910136 added 1 commit 2025-09-21 11:29:48 +08:00
张泊明518370910136 requested changes 2025-09-21 19:32:41 +08:00
joint_teapot/app.py Outdated
@ -169,1 +169,4 @@
@app.command(
"close-issues",
help="close specific issues in a repository: joint-teapot close-issues REPO_NAME ISSUE_NUMBER [ISSUE_NUMBER ...]",
张泊明518370910136 commented 2025-09-21 19:23:31 +08:00
Owner
Copy Link

no need to add sample usage in help text, typer will generate it

no need to add sample usage in help text, typer will generate it
joint_teapot/app.py Outdated
@ -170,0 +175,4 @@
repo_name: str,
issue_numbers: List[int] = Argument(..., help="One or more issue numbers to close"),
dry_run: bool = Option(
False, "--dry-run/--no-dry-run", help="Enable dry run (no changes will be made)"
张泊明518370910136 commented 2025-09-21 19:25:58 +08:00
Owner
Copy Link

"--dry-run/--no-dry-run" is also auto generated by typer

`"--dry-run/--no-dry-run"` is also auto generated by typer
joint_teapot/app.py Outdated
@ -170,0 +219,4 @@
),
repo: bool = Option(
False,
"--repo",
张泊明518370910136 commented 2025-09-21 19:26:13 +08:00
Owner
Copy Link

ditto

ditto
joint_teapot/workers/gitea.py Outdated
@ -548,6 +590,492 @@ class Gitea:
self.create_milestone(repo_name, milestone, description, due_date)
logger.info(f"Created milestone {milestone} in {repo_name}")
def label_issues(
张泊明518370910136 commented 2025-09-21 19:32:34 +08:00
Owner
Copy Link

function body too long

function body too long
joint_teapot/workers/gitea.py Outdated
@ -551,0 +1038,4 @@
path = f"/repos/{self.org_name}/{repo_name}/labels/{enc_name}"
url = f"{self.api_client.configuration.host}{path}"
try:
resp = requests.delete(url, headers=headers_local, timeout=10)
张泊明518370910136 commented 2025-09-21 19:31:26 +08:00
Owner
Copy Link

Does focs_gitea provide this endpoint?

Does `focs_gitea` provide this endpoint?
蔡雨翔524370910013 commented 2025-09-21 22:31:53 +08:00
Author
First-time contributor
Copy Link

focs_gitea seems not to provide the endpoint/api to create labels. Also, using the label issue directly result in a failure on my computer, the issue can't be labeled.

focs_gitea seems not to provide the endpoint/api to create labels. Also, using the label issue directly result in a failure on my computer, the issue can't be labeled.
张泊明518370910136 commented 2025-09-22 07:30:48 +08:00
Owner
Copy Link

Does self.issue_api.issue_create_label and self.issue_api.issue_delete_label work?

Does `self.issue_api.issue_create_label` and `self.issue_api.issue_delete_label` work?
蔡雨翔524370910013 commented 2025-09-22 12:17:55 +08:00
Author
First-time contributor
Copy Link

Sorry, not tried. I thought that if label-issue failed then the other two might be failed. Will refactor to try to use these two tonight.

Sorry, not tried. I thought that if label-issue failed then the other two might be failed. Will refactor to try to use these two tonight.
蔡雨翔524370910013 added 1 commit 2025-09-21 23:20:57 +08:00
蔡雨翔524370910013 commented 2025-09-21 23:21:26 +08:00
Author
First-time contributor
Copy Link

refactored, dry-run issue fixed.

refactored, dry-run issue fixed.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u master:arthurcai-master
git checkout arthurcai-master

Merge

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff arthurcai-master
git checkout master
git merge --ff-only arthurcai-master
git checkout arthurcai-master
git rebase master
git checkout master
git merge --no-ff arthurcai-master
git checkout master
git merge --squash arthurcai-master
git checkout master
git merge --ff-only arthurcai-master
git checkout master
git merge arthurcai-master
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
张泊明518370910136
Labels
Clear labels   
Compat/Breaking

Breaking change that won't be backward compatible

  
Kind/Bug

Something is not working

  
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

  
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

No Label
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 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: JOJ/Joint-Teapot#6
No description provided.
Delete Branch "arthurcai/Joint-Teapot:master"

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?