Skip to content

fix(error): correct 'occured'/'occurr' typos in Error::Io#415

Open
SAY-5 wants to merge 1 commit intoprisma:mainfrom
SAY-5:fix-occured-typo-error-rs
Open

fix(error): correct 'occured'/'occurr' typos in Error::Io#415
SAY-5 wants to merge 1 commit intoprisma:mainfrom
SAY-5:fix-occured-typo-error-rs

Conversation

@SAY-5
Copy link
Copy Markdown

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

src/error.rs had three spelling errors clustered around the Error enum:

  • Doc comment on the enum: might occurr duringmight occur during
  • thiserror error message for Error::Io: An error occured during the attempt of performing I/OAn error occurred ...
  • Doc comment on Error::Io: An error occured when performing I/OAn error occurred when performing I/O

The error message is user-visible whenever a tiberius client logs or formats an Error::Io (it's the Display impl produced by thiserror), so correcting it removes a visible spelling mistake from downstream applications.

cargo check (default features) and cargo check --no-default-features --features rustls,tokio-util,sql-browser-tokio both stay clean against current main.

src/error.rs had three spelling errors clustered around the Error enum:

* Doc comment on the enum: 'might occurr during' -> 'might occur during'
* thiserror error message for Error::Io: 'An error occured during the
  attempt of performing I/O' -> 'An error occurred ...'
* Doc comment on Error::Io: 'An error occured when performing I/O' ->
  'An error occurred when performing I/O'

The error message is user-visible whenever a tiberius client logs or
formats an Io error, so correcting it removes a small but visible
spelling mistake from downstream applications. cargo check on the
default tokio + rustls feature set stays clean.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 15, 2026

Summary by CodeRabbit

  • Documentation
    • Corrected spelling in error messages for improved clarity.

Walkthrough

Spelling corrections in the Error enum: "occurr" changed to "occur" in the module-level comment, the Io variant's error message string, and its documentation comment. No structural changes or logic modifications.

Changes

Cohort / File(s) Summary
Error Documentation
src/error.rs
Corrected spelling "occurr" → "occur" in module comment, Io variant error message, and doc comment.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: correcting spelling typos ('occured'/'occurr' → 'occurred'/'occur') in the Error::Io variant and related documentation in src/error.rs.
Description check ✅ Passed The description is directly related to the changeset, providing specific details about the three spelling corrections, their locations, and verification that cargo check passes.
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 unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/error.rs (1)

86-86: 🧹 Nitpick | 🔵 Trivial

Optional: Fix similar typo in this error message.

While reviewing the spelling corrections, noticed "convertiong" should be "converting" on this line. Consider fixing this similar typo for consistency.

📝 Proposed fix
-        Self::Conversion(format!("Error convertiong a Guid value {}", e).into())
+        Self::Conversion(format!("Error converting a Guid value {}", e).into())
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/error.rs` at line 86, Fix the typo in the Conversion error message in
src/error.rs: replace the string "Error convertiong a Guid value {}" used in
Self::Conversion(...) with "Error converting a Guid value {}" so the error text
is spelled correctly; search for other occurrences of "convertiong" and correct
them similarly to keep messages consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@src/error.rs`:
- Line 86: Fix the typo in the Conversion error message in src/error.rs: replace
the string "Error convertiong a Guid value {}" used in Self::Conversion(...)
with "Error converting a Guid value {}" so the error text is spelled correctly;
search for other occurrences of "convertiong" and correct them similarly to keep
messages consistent.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 626e294d-4b4f-4e02-8613-dc92ebf97c5d

📥 Commits

Reviewing files that changed from the base of the PR and between a6b4fcd and fa4e567.

📒 Files selected for processing (1)
  • src/error.rs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant