Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds Kibana pull and issue attributes #1933

Merged
merged 2 commits into from
Aug 5, 2020

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