Skip to content

[DOCS] Set explicit anchor for "SysV init vs systemd" section #42525

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

Merged
merged 3 commits into from
May 28, 2019
Merged

[DOCS] Set explicit anchor for "SysV init vs systemd" section #42525

merged 3 commits into from
May 28, 2019

Conversation

jrodewig
Copy link
Contributor

This PR attempts to fix the following autogenerated anchors so they render consistently in AsciiDoc and Asciidoctor:

  • (5.0 - 6.2) _sysv_literal_init_literal_vs_literal_systemd_literal_2 -> _sysv_init_vs_systemd_2
  • (5.0 - 6.2) _sysv_literal_init_literal_vs_literal_systemd_literal -> _sysv_init_vs_systemd

However, the init-systemd.asciidoc file containing the anchor is included in two other files:

  • deb.asciidoc
  • rpm.asciidoc

This means the explicit anchors is reused. Reusing the anchor results in the following errors:

INFO:build_docs:asciidoctor: WARNING: setup/install/init-systemd.asciidoc: line 2: id assigned to section already in use: _sysv_literal_init_literal_vs_literal_systemd_literal

Next steps

@nik9000
Let me know if you have any suggestions. We might be better off just accepting the anchor change.

@jrodewig jrodewig added >docs General docs changes :Docs labels May 24, 2019
@jrodewig jrodewig requested a review from nik9000 May 24, 2019 14:18
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@nik9000
Copy link
Member

nik9000 commented May 24, 2019

I think the anchor change is better than duplicating the anchor.

@jrodewig
Copy link
Contributor Author

Thanks for your help @nik9000. This is now ready for your review.

@jrodewig jrodewig marked this pull request as ready for review May 28, 2019 15:22
@@ -128,6 +128,8 @@ sudo dpkg -i elasticsearch-{version}.deb

endif::[]

:int-context: deb
Copy link
Member

Choose a reason for hiding this comment

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

Could you do :init-context: instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added with c161040. Thanks!

Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@jrodewig jrodewig merged commit fb2b81f into elastic:6.2 May 28, 2019
@jrodewig jrodewig deleted the asciidoctor-sysv-vs-systemd branch May 28, 2019 19:16
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
jrodewig added a commit that referenced this pull request May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants