-
Notifications
You must be signed in to change notification settings - Fork 197
Add sheet2docs automation script and workflow #4729
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
Conversation
✅ Vale Linting ResultsNo issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
…ntent into add-sheet2csv-automation
Mpdreamz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these be moved to .github/scripts ?
Since the root folders are all docs.
|
Ahh there is prior art: https://github.com/elastic/docs-content/tree/main/scripts Still would be nice to have these in |
|
@Mpdreamz Want to do that in a follow-up PR if that's OK. Could you approve so we merge this one? |
…ntent into add-sheet2csv-automation
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just added a few suggestions
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Summary
Contributes to https://github.com/elastic/docs-content-internal/issues/654
Generative AI disclosure
Tool(s) and model(s) used: Claude Opus 4.5 in Cursor