Skip to content

e2e/eventstest: fix 'occured' -> 'occurred' in handler.go log message#1156

Open
SAY-5 wants to merge 1 commit intomattermost:masterfrom
SAY-5:fix-handler-occured-typo
Open

e2e/eventstest: fix 'occured' -> 'occurred' in handler.go log message#1156
SAY-5 wants to merge 1 commit intomattermost:masterfrom
SAY-5:fix-handler-occured-typo

Conversation

@SAY-5
Copy link
Copy Markdown

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

Log format string in e2e/pkg/eventstest/handler.go line 197 reads events occured. Fixed to occurred. String-literal-only change.

NONE

Log format string in e2e/pkg/eventstest/handler.go line 197 read 'events occured'. Fixed to 'occurred'. String-literal-only change.

Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@mm-cloud-bot
Copy link
Copy Markdown

@SAY-5: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

I understand the commands that are listed here

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

📝 Walkthrough

Walkthrough

A spelling correction was made to an error message string in the events test handler, changing "occured" to "occurred". No functional or logic changes were introduced.

Changes

Cohort / File(s) Summary
Typo Fix
e2e/pkg/eventstest/handler.go
Corrected spelling in error message: "occured" → "occurred"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a typo in a log message from 'occured' to 'occurred' in the handler.go file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description accurately describes the changeset: a single-line correction of a typo in a log format string from 'occured' to 'occurred' in e2e/pkg/eventstest/handler.go.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Copy link
Copy Markdown
Author

SAY-5 commented Apr 16, 2026

NONE

@mm-cloud-bot mm-cloud-bot added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed labels Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note-none Denotes a PR that doesn't merit a release note.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants