-
Notifications
You must be signed in to change notification settings - Fork 190
Add missing page-level applies_to for ingest docs
#3238
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
b2593ff to
07769aa
Compare
07769aa to
11eed7d
Compare
11eed7d to
3ff46ce
Compare
3ff46ce to
b70b4ea
Compare
Vale Linting ResultsSummary: 1 warning, 5 suggestions found
|
| File | Line | Rule | Message |
|---|---|---|---|
| reference/fleet/mutual-tls.md | 148 | Elastic.DontUse | Don't use 'at this point'. |
💡 Suggestions (5)
| File | Line | Rule | Message |
|---|---|---|---|
| reference/fleet/agent-command-reference.md | 269 | Elastic.Acronyms | 'SHA' has no definition. |
| reference/fleet/agent-command-reference.md | 661 | Elastic.Acronyms | 'SHA' has no definition. |
| reference/fleet/grant-access-to-elasticsearch.md | 17 | Elastic.Wordiness | Consider using 'to' instead of 'in order to'. |
| reference/fleet/mutual-tls.md | 148 | Elastic.Wordiness | Consider using 'because' instead of 'Since'. |
| reference/fleet/mutual-tls.md | 191 | Elastic.WordChoice | Consider using 'can, might' instead of 'may', unless the term is in the UI. |
8d9bc46 to
f6f33ba
Compare
f6f33ba to
5cca8ea
Compare
5cca8ea to
1ff9569
Compare
78b4f46 to
92f0d63
Compare
57c3bc2 to
8ad29a1
Compare
8ad29a1 to
796e752
Compare
karenzone
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.
Left comments and suggestions inline for Configure Standalone Agents
karenzone
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.
Left comments and suggestions inline
|
@vishaangelova, I made more than one review pass to not risk GitHub timing out and flushing my comments. Please let me know if you'd like to get together and discuss. |
eb80e00 to
7fb03b6
Compare
karenzone
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.
@vishaangelova, we've addressed concerns and have plans in place to continue iterating with tooling updates coming soon. So for now, LGTM. 🎉
|
@bmorelli25, now that Karen has approved, do you think we can merge this? I've addressed your comments in d0dd048, 233a90f, and 796e752. Let me know if I missed something and I’ll have a look. If not, feel free to merge! |
bmorelli25
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.
🚀 nice job! And a big thank you to all of the reviewers as well!
Summary
This PR adds/updates page-level
applies_toinformation for:manage-data/ingestfolderreference/apm-agents/index.md(+ adds some links)reference/fleetfolder:index.md)reference/ingestion-tools/index.md(+ adds some links)Generative AI disclosure
Cursor / gpt-5.1 model
Review assignments