Skip to content

Commit

Permalink
Merge pull request #21821: Do not allow postcommit jobs phrase trigge…
Browse files Browse the repository at this point in the history
…ring
  • Loading branch information
kennknowles authored Jun 13, 2022
2 parents 3356cc6 + 85fcf98 commit 87a7dcc
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions .test-infra/jenkins/PostcommitJobBuilder.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class PostcommitJobBuilder {
* Set the job details.
*
* @param nameBase Job name for the postcommit job, a _PR suffix added if the trigger is set.
* @param triggerPhrase Phrase to trigger jobs, empty to not have a trigger.
* @param triggerPhrase Phrase to trigger jobs, IGNORED.
* @param githubUiHint Short description in the github UI.
* @param scope Delegate for the job.
* @param jobDefinition Closure for the job.
Expand All @@ -51,9 +51,6 @@ class PostcommitJobBuilder {
jobDefinition = {}) {
PostcommitJobBuilder jb = new PostcommitJobBuilder(scope, jobDefinition)
jb.defineAutoPostCommitJob(nameBase)
if (triggerPhrase) {
jb.defineGhprbTriggeredJob(nameBase + "_PR", triggerPhrase, githubUiHint, false)
}
}

void defineAutoPostCommitJob(name) {
Expand Down

0 comments on commit 87a7dcc

Please sign in to comment.