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

Remove x-pack docs dir from ES book #2753

Merged
merged 3 commits into from
Sep 13, 2023
Merged

Remove x-pack docs dir from ES book #2753

merged 3 commits into from
Sep 13, 2023

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Sep 7, 2023

Problem: With elastic/elasticsearch#99209, the elasticsearch repo's x-pack/docs directory no longer contains docs for the Elasticsearch Guide in 8.10+. However, the docs build config still lists the x-pack/docs directory as a dependency.

Solution: Update the docs build config to remove the x-pack/docs dependency for the Elasticsearch Guide in 8.10+.

Depends on elastic/elasticsearch#99209
Rel: https://github.com/elastic/platform-docs-team/issues/208

@jrodewig jrodewig marked this pull request as ready for review September 12, 2023 23:07
@jrodewig jrodewig requested review from a team and abdonpijpelink September 12, 2023 23:07
@jrodewig jrodewig merged commit dbfdefc into elastic:master Sep 13, 2023
@jrodewig jrodewig deleted the jrodewig/remove-x-pack-dir branch September 13, 2023 08:43
jrodewig added a commit to elastic/elasticsearch that referenced this pull request Sep 13, 2023
**Problem:** With #99209, the elasticsearch repo's `x-pack/docs` directory no longer contains docs for the Elasticsearch Guide in 8.10+.

The `gradle.build` files in the directory were only left in so that the docs build doesn't break. That's been fixed with elastic/docs#2753.

**Solution:** Removes the `x-pack/docs` directory.

Depends on #99209, elastic/docs#2753
Closes: https://github.com/elastic/platform-docs-team/issues/208
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.

1 participant