Skip to content

docs: fix 'ElasticSearch' casing and 'a SSL' -> 'an SSL' across reference docs#50281

Merged
alexandra5000 merged 3 commits intoelastic:mainfrom
MukundaKatta:fix/grammar-terminology-issues
May 7, 2026
Merged

docs: fix 'ElasticSearch' casing and 'a SSL' -> 'an SSL' across reference docs#50281
alexandra5000 merged 3 commits intoelastic:mainfrom
MukundaKatta:fix/grammar-terminology-issues

Conversation

@MukundaKatta
Copy link
Copy Markdown
Contributor

Fixes #50195.

Picks up the unambiguous items from the audit:

1. Product-name casing (2 files): ElasticSearchElasticsearch in the community-beats pages. Canonical name per Elastic style.

2. Article grammar (12 lines across 6 beats): not a SSL problemnot an SSL problem in the ssl-client-fails.md page for auditbeat / filebeat / heartbeat / metricbeat / packetbeat / winlogbeat. Same 2-line change in each beat (lines 61 + 66).

Docs-only; no schema or code changes.

@MukundaKatta MukundaKatta requested a review from a team as a code owner April 22, 2026 23:56
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • /test : Run the Buildkite pipeline.

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 22, 2026

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @MukundaKatta? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dfba06d0-834e-4dad-9521-0228fc18de7f

📥 Commits

Reviewing files that changed from the base of the PR and between f2ca1ee and 897cb1a.

📒 Files selected for processing (8)
  • docs/extend/community-beats.md
  • docs/reference/auditbeat/ssl-client-fails.md
  • docs/reference/filebeat/ssl-client-fails.md
  • docs/reference/heartbeat/ssl-client-fails.md
  • docs/reference/libbeat/community-beats.md
  • docs/reference/metricbeat/ssl-client-fails.md
  • docs/reference/packetbeat/ssl-client-fails.md
  • docs/reference/winlogbeat/ssl-client-fails.md

📝 Walkthrough

Walkthrough

This pull request updates documentation across eight files with minor text corrections. Two files (docs/extend/community-beats.md and docs/reference/libbeat/community-beats.md) correct the product name from "ElasticSearch" to "Elasticsearch". Six SSL documentation files (docs/reference/auditbeat/ssl-client-fails.md, docs/reference/filebeat/ssl-client-fails.md, docs/reference/heartbeat/ssl-client-fails.md, docs/reference/metricbeat/ssl-client-fails.md, docs/reference/packetbeat/ssl-client-fails.md, and docs/reference/winlogbeat/ssl-client-fails.md) update the grammatical article from "a SSL" to "an SSL" in their explanatory sections. No functional changes or structural modifications are introduced.

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive PR implements two of four requested text fix categories: ElasticSearch→Elasticsearch (2 files) and a SSL→an SSL (6 files). Omits node_stats a Elasticsearch and overcommitted spelling fixes. Clarify scope: confirm whether partial implementation of issue #50195 is intentional or if remaining fixes (node_stats, overcommitted) should be included in this PR or separate issues.
✅ Passed checks (1 passed)
Check name Status Explanation
Out of Scope Changes check ✅ Passed All changes align with issue #50195 scope: product-name casing and SSL article grammar fixes only. No unrelated modifications detected.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@alexandra5000 alexandra5000 merged commit 3ee3a15 into elastic:main May 7, 2026
2 of 4 checks passed
@vishaangelova vishaangelova added docs Team:Docs Label for the Observability docs team skip-changelog backport-9.4 labels May 7, 2026
@botelastic botelastic Bot removed the needs_team Indicates that the issue/PR needs a Team:* label label May 7, 2026
vishaangelova pushed a commit that referenced this pull request May 7, 2026
… (#50281) (#50535)

(cherry picked from commit 3ee3a15)

Co-authored-by: Mukunda Rao Katta <mukunda.vjcs6@gmail.com>
Co-authored-by: Aleksandra Spilkowska <96738481+alexandra5000@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-9.4 docs skip-changelog Team:Docs Label for the Observability docs team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[text-auditor] Fix clear user-facing grammar/terminology issues in SSL and Elasticsearch docs

3 participants