Skip to content

chore: typo sweep (succesful, occured)#50251

Closed
SAY-5 wants to merge 2 commits into
elastic:mainfrom
SAY-5:chore/typo-sweep
Closed

chore: typo sweep (succesful, occured)#50251
SAY-5 wants to merge 2 commits into
elastic:mainfrom
SAY-5:chore/typo-sweep

Conversation

@SAY-5

@SAY-5 SAY-5 commented Apr 22, 2026

Copy link
Copy Markdown

Two single-character typo fixes in comments/prose:

  • libbeat/processors/dissect/processor_test.go:287 — t.Run subtest name succesful -> successful
  • docs/reference/metricbeat/exported-fields-oracle.md:131 — oracle metricset parse description occured -> occurred

No functional change.

- libbeat/processors/dissect/processor_test.go: test name "succesful" -> "successful"
- docs/reference/metricbeat/exported-fields-oracle.md: "occured" -> "occurred"

No functional change.
@SAY-5 SAY-5 requested review from a team as code owners April 22, 2026 09:36
@SAY-5 SAY-5 requested review from AndersonQ and belimawr April 22, 2026 09:36
@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 mergify Bot assigned SAY-5 Apr 22, 2026
@mergify

mergify Bot commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @SAY-5? 🙏.
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

coderabbitai Bot commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

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: 42e7ffaf-0402-4a51-aa66-186263f07bfe

📥 Commits

Reviewing files that changed from the base of the PR and between 8b7236f and b71069a.

📒 Files selected for processing (2)
  • docs/reference/metricbeat/exported-fields-oracle.md
  • libbeat/processors/dissect/processor_test.go

📝 Walkthrough

Walkthrough

This pull request contains two spelling corrections across documentation and test files. In the Oracle metricbeat exported fields documentation, the word "occured" is corrected to "occurred" in a field description. In the dissect processor test file, a test sub-case name string is corrected from "succesful" to "successful". Both changes are documentation and test metadata corrections with no functional code modifications.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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.

@SAY-5

SAY-5 commented Apr 24, 2026

Copy link
Copy Markdown
Author

The check / fragments jobs are failing on this PR because it's a pure comment / user-facing-string typo sweep (succesfulsuccessful, occuredoccurred) with no behaviour change. Could a maintainer apply whichever label corresponds to "skip changelog fragment" for docs-only touches, or advise which fragment template is appropriate here? Happy to commit the right fragment yaml directly if that's preferred.

@pierrehilbert pierrehilbert added Team:Docs Label for the Observability docs team Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team labels Apr 24, 2026
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@botelastic botelastic Bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Apr 24, 2026
@SAY-5

SAY-5 commented Apr 28, 2026

Copy link
Copy Markdown
Author

Closing — typo-only PR. Multiple maintainers across the OSS ecosystem have flagged my recent typo-sweep PRs as AI-generated spam. Apologies for the noise.

@SAY-5 SAY-5 closed this Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Docs Label for the Observability docs team Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants