Skip to content

Conversation

@KOTungseth
Copy link
Contributor

@KOTungseth KOTungseth commented Aug 5, 2020

Added the following attributes:
:kibana-issue: {kib-issue}
:kibana-pull: {kib-pull}

@KOTungseth KOTungseth requested a review from lcawl August 5, 2020 21:19
@KOTungseth KOTungseth self-assigned this Aug 5, 2020
@gtback
Copy link
Member

gtback commented Aug 5, 2020

Do we need both kib-pull and kibana-pull?

:kib-repo: https://github.com/elastic/kibana/
:kib-issue: {kib-repo}issues/
:kib-pull: {kib-repo}pull/
:kibana-pull: {kib-repo}pull/
Copy link
Contributor

Choose a reason for hiding this comment

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

Not mandatory, but you could make the dependency between these attributes explicit by doing this:

Suggested change
:kibana-pull: {kib-repo}pull/
:kibana-pull: {kib-pull}

@KOTungseth KOTungseth changed the title Adds Kibana pull attribute Adds Kibana pull and issue attributes Aug 5, 2020
@lcawl
Copy link
Contributor

lcawl commented Aug 5, 2020

Do we need both kib-pull and kibana-pull?

If not, we must go back and fix any usage of "kib-pull". For example, I see it in the 7.8.0 release notes.

@lcawl
Copy link
Contributor

lcawl commented Aug 5, 2020

elastic/kibana#74306 builds successfully with these changes

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM!

@KOTungseth KOTungseth merged commit 12c042a into elastic:master Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants