-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/invite after cherry pick pr created #1011
Feature/invite after cherry pick pr created #1011
Conversation
…ethod from `handle` method
…thod from `handle` method
…ethod from `handle` method
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
🎊 PR Preview 9b13d8b has been successfully built and deployed to https://ti-community-infra-tichi-preview-pr-1011.surge.sh 🕐 Build time: 0.017s 🤖 By surge-preview |
/cc @Mini256 |
Codecov Report
@@ Coverage Diff @@
## master #1011 +/- ##
==========================================
+ Coverage 80.39% 80.51% +0.11%
==========================================
Files 20 21 +1
Lines 3265 3321 +56
==========================================
+ Hits 2625 2674 +49
- Misses 413 418 +5
- Partials 227 229 +2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also update the help provider and docs.
internal/pkg/externalplugins/cherrypicker/cherrypicker_invite_test.go
Outdated
Show resolved
Hide resolved
internal/pkg/externalplugins/cherrypicker/cherrypicker_invite_test.go
Outdated
Show resolved
Hide resolved
internal/pkg/externalplugins/cherrypicker/cherrypicker_invite_test.go
Outdated
Show resolved
Hide resolved
- replace type of `permission` from `string` to `RepoPermissionLevel`.
internal/pkg/externalplugins/cherrypicker/cherrypicker_invite.go
Outdated
Show resolved
Hide resolved
@Mini256 I updated the documents, please take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔 Maybe we should limit only cherry-pick pr can handle the /cherry-pick-invite
command.
Rest LGTM. Well Done! 👍
And please make full tests in the test-dev repo.
@@ -166,6 +169,14 @@ func HelpProvider(epa *tiexternalplugins.ConfigAgent) externalplugins.ExternalPl | |||
WhoCanUse: "Members of the trusted organization for the repo or anyone(depends on the AllowAll configuration).", | |||
Examples: []string{"/cherrypick release-3.9", "/cherry-pick release-1.15"}, | |||
}) | |||
pluginHelp.AddCommand(pluginhelp.Command{ | |||
Usage: cherryPickInviteExample, | |||
Description: "Request a collaborator invitation" + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description: "Request a collaborator invitation" + | |
Description: "Request a collaborator invitation (Only works in cherry-pick pull requests)." + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Next PR
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 9b13d8b
|
{"job":{"type":"presubmit","job":"pull-tichi-rerere","buildid":"1572421727613358080","prowjobid":"3f1a3f07-395a-11ed-bc19-8ef6841f76ec","refs":{"org":"ti-community-infra","repo":"tichi","repo_link":"https://github.com/ti-community-infra/tichi","base_ref":"master","base_sha":"10cfcccaa57db0f5f278017d1551dcb4c788a04d","base_link":"https://github.com/ti-community-infra/tichi/commit/10cfcccaa57db0f5f278017d1551dcb4c788a04d","pulls":[{"number":1011,"author":"wuhuizuo","sha":"9b13d8b4c3b9dfb437e7b59c4a435e16a68f159d","title":"Feature/invite after cherry pick pr created","link":"https://github.com/ti-community-infra/tichi/pull/1011","commit_link":"https://github.com/ti-community-infra/tichi/pull/1011/commits/9b13d8b4c3b9dfb437e7b59c4a435e16a68f159d","author_link":"https://github.com/wuhuizuo"}]},"decoration_config":{"utility_images":{"clonerefs":"gcr.io/k8s-prow/clonerefs:v20211229-5ea9b96f66","initupload":"gcr.io/k8s-prow/initupload:v20211229-5ea9b96f66","entrypoint":"gcr.io/k8s-prow/entrypoint:v20211229-5ea9b96f66","sidecar":"gcr.io/k8s-prow/sidecar:v20211229-5ea9b96f66"},"gcs_configuration":{"bucket":"gs://prow-tidb-logs","path_strategy":"explicit"},"gcs_credentials_secret":"gcs-credentials"}},"options":{"retesting-branch":"rerere","retry":3,"require_contexts":{},"timeout":900000000000},"current_retry_times":1,"time":"2022-09-21T03:08:34.707741727Z"}
No description provided.