Skip to content

Conversation

@craigtaverner
Copy link
Contributor

The PR at #124312 created a lot of generated docs in a new location. This PR updates the .gitattributes file to hide these in the new locations.

@craigtaverner craigtaverner added >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Mar 13, 2025
@craigtaverner craigtaverner requested a review from ivancea March 13, 2025 13:22
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@craigtaverner craigtaverner mentioned this pull request Mar 13, 2025
21 tasks
Copy link
Contributor

@ivancea ivancea left a comment

Choose a reason for hiding this comment

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

:shipit:

docs/reference/esql/functions/*/** linguist-generated=true
# ESQL functions docs are autogenerated. More information at `docs/reference/query-languages/esql/README.md`
docs/reference/query-languages/esql/_snippets/functions/*/** linguist-generated=true
#docs/reference/query-languages/esql/_snippets/operators/*/** linguist-generated=true
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this not ready yet? I would maybe add it anyway, it will just be collapsed by default. A to not open more PRs for this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The operators are still mostly manually created. I thought this setting would hide these changes from PRs and we should not do that for manually edited files. But I hope we can fix this moving forward. Certainly the previous PR took a reasonable step closer to automating operator docs generating. Previously only the types list was generated. Now we can also generate detailedDescription, notes and examples. But there are a few parts missing and a few operators that are 100% manual docs.

@craigtaverner craigtaverner merged commit a7d1bd8 into elastic:main Mar 13, 2025
17 checks passed
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Mar 13, 2025
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Mar 13, 2025
elasticsearchmachine pushed a commit that referenced this pull request Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants