Skip to content

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Mar 26, 2022

@staabm
Copy link
Contributor

staabm commented Mar 26, 2022

Please reference the relevant issues in the PR description like
"closes $link-to-issue" (each on a separate line)

@VincentLanglet
Copy link
Contributor Author

Closes phpstan/phpstan#6576
Closes phpstan/phpstan#6842

But they are already closed

@staabm
Copy link
Contributor

staabm commented Mar 26, 2022

Even if they are already closed - adding the link - will show up a reference at the issue.

So it eases getting things together later on.

@ondrejmirtes ondrejmirtes merged commit 1d283fc into phpstan:1.5.x Mar 27, 2022
@ondrejmirtes
Copy link
Member

Awesome, thank you.

@VincentLanglet
Copy link
Contributor Author

Awesome, thank you.

I can do more if you need to :)

@VincentLanglet VincentLanglet deleted the moreRegressionTests branch March 27, 2022 17:00
@ondrejmirtes
Copy link
Member

I'll keep that in mind when it happens next time - when some bugs are fixed as a side effect of another fix and @phpstan-bot discovers that 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants