Skip to content

Conversation

@butonic
Copy link
Member

@butonic butonic commented Aug 18, 2016

fixes a bug introduced with #25845

cc @PVince81 @jvillafanez I will look for a test case to cover this in a separate PR

@mention-bot
Copy link

@butonic, thanks for your PR! By analyzing the annotation information on this pull request, we identified @Xenopathic, @PVince81 and @icewind1991 to be potential reviewers

@butonic butonic added this to the 9.1.1 milestone Aug 18, 2016
@butonic butonic changed the title fix typo [stable9.1] fix typo Aug 18, 2016
@PVince81
Copy link
Contributor

👍

@jvillafanez
Copy link
Member

🚑 👍

@PVince81
Copy link
Contributor

this wasn't discovered by CI and likely won't either.

A local run of ./autotest-external.sh which covers some cases didn't show any failures.

@PVince81 PVince81 merged commit d4ec50c into stable9.1 Aug 18, 2016
@PVince81 PVince81 deleted the stable9.1-fixtypo branch August 18, 2016 09:51
@DeepDiver1975
Copy link
Member

A local run of ./autotest-external.sh which covers some cases didn't show any failures.

is executed by jenkins as well

@lock
Copy link

lock bot commented Aug 4, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants