Skip to content

Added DSL documentation to Guide #2761

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
Jan 24, 2025

Conversation

miguelgrinberg
Copy link
Contributor

@miguelgrinberg miguelgrinberg commented Jan 23, 2025

This adds the DSL documentation (not including the reference sections) to the documentation guide.

The contents are mostly translated from the original RST sources, without any major changes. There are improvements that can be made to the content as well, such as showing the new typed syntax in code blocks, which I'll address separately.

Copy link

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@miguelgrinberg miguelgrinberg force-pushed the dsl-added-to-guide branch 3 times, most recently from 5bd71ce to 3a03b40 Compare January 23, 2025 16:30
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! This literally looks great.

Comment on lines +45 to +48
include::dsl/configuration.asciidoc[]
include::dsl/tutorials.asciidoc[]
include::dsl/howto.asciidoc[]
include::dsl/examples.asciidoc[]
Copy link
Member

Choose a reason for hiding this comment

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

Can we also add links to those sections? On mobile the navigation is at the bottom of the page and easy to miss.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree that it would be good to add these links, but the way this is currently implemented is hacky. The links that appear in the body (as opposite to those that are in the sidebar) are implemented in an HTML file, not an asciidoc file. The only links that are currently in the body are links to top-level sections. I'm not sure there is a robust way of adding links to sub-sections, since these links are generated by asciidoc.

I'm not going to implement this right now, I think we should wait until the docs migrate and then we can see what can be done within the new docs system.

@miguelgrinberg miguelgrinberg merged commit 8a27080 into elastic:main Jan 24, 2025
12 checks passed
@miguelgrinberg miguelgrinberg deleted the dsl-added-to-guide branch January 24, 2025 18:39
github-actions bot pushed a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)
github-actions bot pushed a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)
miguelgrinberg added a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)
miguelgrinberg added a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)
miguelgrinberg added a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)

Co-authored-by: Miguel Grinberg <miguel.grinberg@gmail.com>
miguelgrinberg added a commit that referenced this pull request Feb 11, 2025
* Added DSL documentation to Guide

* clarify that this is a Python DSL

(cherry picked from commit 8a27080)

Co-authored-by: Miguel Grinberg <miguel.grinberg@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants