Skip to content

Conversation

@pickypg
Copy link
Member

@pickypg pickypg commented Apr 26, 2019

@pickypg pickypg requested a review from ycombinator April 26, 2019 19:04
@ycombinator
Copy link
Contributor

PR description is incorrect but PR contents are correct.

@pickypg
Copy link
Member Author

pickypg commented Apr 26, 2019

Yeah, I cherry picked the 7.x backport commit. I'll rename it and push that so that CI wakes up for ci/2.

…lastic#40899)

Backports the usage change to 7.0. A separate backport is needed to
change the version for 6.7, which will complete this backport.
@pickypg pickypg force-pushed the backport/7.0-monitoring-template-version branch from 65de660 to 40f39ce Compare April 26, 2019 19:16
@pickypg
Copy link
Member Author

pickypg commented Apr 26, 2019

PR description is incorrect but PR contents are correct.

Oh, woops! I copied my backport format from a previous backport and apparently left the old header in the bulleted list. Good catch.

Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@pickypg pickypg merged commit 94211dc into elastic:7.0 Apr 26, 2019
@pickypg pickypg deleted the backport/7.0-monitoring-template-version branch April 26, 2019 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants