Skip to content

Commit

Permalink
Merge pull request #1561 from spring-financial-group/reduce-noise
Browse files Browse the repository at this point in the history
fix: stop link in comment creating reference to issue
  • Loading branch information
jenkins-x-bot authored Jul 12, 2023
2 parents 048aadf + 955d616 commit 8b483bb
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions pkg/plugins/trigger/pull-request.go
Original file line number Diff line number Diff line change
Expand Up @@ -144,12 +144,11 @@ func infoMsg(c Client, pr scm.PullRequest) error {
return nil
}

org, repo, a := orgRepoAuthor(pr)
author := string(a)
org, repo, _ := orgRepoAuthor(pr)

comment := fmt.Sprintf(`[jx-info] Hi @%s. We've detected that the pipelines in this repository are using a syntax that will soon be deprecated.
We'll continue to update you through PRs as we progress. Please check [#8589](https://github.com/jenkins-x/jx/issues/8589) for further information.
`, author)
comment := `[jx-info] Hi, we've detected that the pipelines in this repository are using a syntax that will soon be deprecated.
We'll continue to update you through PRs as we progress. Please check [#8589](https://github.com/jenkins-x/jx/issues/8589) for further information.
`

if err := c.SCMProviderClient.CreateComment(org, repo, pr.Number, true, comment); err != nil {
return errors.Wrap(err, "failed to comment info message")
Expand Down

0 comments on commit 8b483bb

Please sign in to comment.