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

[DOCS] Add safeguard directory attributes #57628

Closed
wants to merge 1 commit into from
Closed

[DOCS] Add safeguard directory attributes #57628

wants to merge 1 commit into from

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Jun 3, 2020

PRs #57390 and #57489 swapped all instances of the {docdir} and
{asciidoc-dir} attributes to {es-repo-dir}. Using {es-repo-dir}
exclusively prevents issues when content from the ES refernce is reused
in other doc books.

This adds related safeguard attributes, which fall back to {es-repo-dir}. This
should curb future problems if a contributor accidentally uses {docdir} or
{asciidoc-dir}.

PRs #57390 and #57489 swapped all instances of the `{docdir}` and
`{asciidoc-dir}` attributes to `{es-repo-dir}`. Using `{es-repo-dir}`
exclusively prevents issues when content from the ES refernce is reused
in other doc books.

This adds related safeguard attributes, which fall back to `{es-repo-dir}`. This
should curb future problems if a contributor accidentally uses `{docdir}` or
`{asciidoc-dir}`.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Jun 3, 2020
@jrodewig
Copy link
Contributor Author

jrodewig commented Jun 4, 2020

Turns out this won't help downstream books reusing the repo so I'm closing this one.

@jrodewig jrodewig closed this Jun 4, 2020
@jrodewig jrodewig deleted the docs__add-safeguard-es-repo-attributes branch June 6, 2020 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes Team:Docs Meta label for docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants