Skip to content

Commit

Permalink
Merge pull request #926 from kkmuffme/improve-bot-text
Browse files Browse the repository at this point in the history
Improve bot text vimeo/psalm#10376
  • Loading branch information
weirdan authored Jan 31, 2024
2 parents 3878ae6 + 2d69aa9 commit 5f391d7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/Responder/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { ResolvedSnippet, SnippetIssue } from "../SnippetResolver"
export class Responder {

greet(login: string) {
return `Hey @${login}, can you reproduce the issue on https://psalm.dev ?`
return `Hey @${login}, can you reproduce the issue on https://psalm.dev? These will be used as phpunit tests when implementing the feature or fixing this bug.`
}

shouldGreet(issue: string, repoName: string) {
Expand Down Expand Up @@ -62,7 +62,7 @@ ${snippetOutput}
if (snippet.internalError !== null) {
return this.formatInternalError(snippet)
} else if (snippet.results !== null) {
return this.formatSnippetResult(snippet)
return this.formatSnippetResult(snippet)
}
return '';
}
Expand Down
2 changes: 1 addition & 1 deletion test/Responder.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ describe('Responder', () => {
describe('Actual greeting', () => {
test('Greeting text', () => {
expect(responder.greet('aUser'))
.toEqual('Hey @aUser, can you reproduce the issue on https://psalm.dev ?')
.toEqual('Hey @aUser, can you reproduce the issue on https://psalm.dev? These will be used as phpunit tests when implementing the feature or fixing this bug.')
})
})

Expand Down

0 comments on commit 5f391d7

Please sign in to comment.