Skip to content

chore: fix typos in comments and log messages#37715

Open
sid2487 wants to merge 6 commits intoRocketChat:developfrom
sid2487:fix/spelling-comments
Open

chore: fix typos in comments and log messages#37715
sid2487 wants to merge 6 commits intoRocketChat:developfrom
sid2487:fix/spelling-comments

Conversation

@sid2487
Copy link

@sid2487 sid2487 commented Dec 8, 2025

Summary

This PR fixes small spelling mistakes in comments, logs, and user-facing messages.
These changes are non-functional and do not affect code behavior.

Changes

  • occured → occurred
  • recieved → received

Files updated

  • apps/meteor/app/meteor-accounts-saml/server/lib/SAML.ts
  • ee/packages/media-calls/src/sip/providers/IncomingSipCall.ts
  • ee/packages/media-calls/src/sip/providers/OutgoingSipCall.ts
  • packages/apps-engine/src/definition/messages/IMessageReactionContext.ts
  • packages/apps-engine/src/server/AppManager.ts

Notes

  • No API or functional changes.
  • The typo fix in the mailer API (adresses) was intentionally not included to avoid potential breaking changes.

Summary by CodeRabbit

  • Chores
    • Corrected spelling in user-facing error messages and operational logs to improve clarity and professionalism across several components.
  • Documentation
    • Fixed typos in developer-facing documentation comments to improve readability of API docs and inline docs.

✏️ Tip: You can customize this high-level summary in your review settings.

@sid2487 sid2487 requested a review from a team as a code owner December 8, 2025 06:05
@changeset-bot
Copy link

changeset-bot bot commented Dec 8, 2025

⚠️ No Changeset found

Latest commit: 9e51c71

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Dec 8, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@CLAassistant
Copy link

CLAassistant commented Dec 8, 2025

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 8, 2025

Walkthrough

Spelling corrections across five files: "occured" → "occurred" in error messages/comments and "recieved" → "received" in JSDoc. No behavioral, API, or exported-entity changes.

Changes

Cohort / File(s) Summary
SAML Error Messages
apps/meteor/app/meteor-accounts-saml/server/lib/SAML.ts
Corrected typo in user-facing error HTML: "occured" → "occurred".
SIP Call Error & Log Messages
ee/packages/media-calls/src/sip/providers/IncomingSipCall.ts, ee/packages/media-calls/src/sip/providers/OutgoingSipCall.ts
Fixed typo in logged error messages: "occured" → "occurred".
App Manager & Engine Definitions
packages/apps-engine/src/server/AppManager.ts, packages/apps-engine/src/definition/messages/IMessageReactionContext.ts
Corrected typos in comment and JSDoc: "occured" → "occurred", "recieved" → "received".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

  • KevLehman
  • lucas-a-pelegrino
  • tassoevan

Poem

🐰 I hopped through code both near and far,
Found letters astray like a tiny star,
I nudged them gently, fixed each little plight,
Now "occurred" and "received" sit snug and right. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: fix typos in comments and log messages' accurately and clearly describes the main purpose of the PR—correcting spelling errors across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@d-gubert d-gubert changed the title chore(spelling): fix typos in comments and log messages chore: fix typos in comments and log messages Jan 22, 2026
@ggazzo ggazzo added this to the 8.3.0 milestone Feb 26, 2026
@ggazzo ggazzo added the stat: QA assured Means it has been tested and approved by a company insider label Feb 26, 2026
@dionisio-bot dionisio-bot bot enabled auto-merge February 26, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants