Skip to content

fix: correct 'occured' typo in I/O error message#423

Open
DucMinhNe wants to merge 1 commit into
prisma:mainfrom
DucMinhNe:fix/occured-typo-io-error
Open

fix: correct 'occured' typo in I/O error message#423
DucMinhNe wants to merge 1 commit into
prisma:mainfrom
DucMinhNe:fix/occured-typo-io-error

Conversation

@DucMinhNe

Copy link
Copy Markdown

Summary

Fixes a typo occuredoccurred in the Io error variant of src/error.rs.

The fix touches the user-facing Display string produced by the #[error(...)] attribute (shown to anyone who prints or logs this error), plus the matching doc comment one line above:

#[error("An error occurred during the attempt of performing I/O: {}", message)]
/// An error occurred when performing I/O to the server.

No behavior change — spelling only.

@coderabbitai

coderabbitai Bot commented Jun 2, 2026

Copy link
Copy Markdown

Review Change Stack

Summary by CodeRabbit

  • Bug Fixes
    • Fixed spelling errors in system error messages for improved readability.

Walkthrough

This PR corrects a spelling error in the I/O error variant's display message. The #[error(...)] text for Error::Io was updated from the misspelled "occured" to the correct spelling "occurred". No error types, variant shapes, or conversion logic were changed.

Changes

Error Message Spelling Fix

Layer / File(s) Summary
I/O error message spelling fix
src/error.rs
The #[error(...)] display string for the Error::Io variant is corrected from "occured" to "occurred".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: correct 'occured' typo in I/O error message' accurately and specifically describes the main change: a spelling correction from 'occured' to 'occurred' in an I/O error message.
Description check ✅ Passed The description is directly related to the changeset, providing clear context about the typo fix in the I/O error variant and specifying the exact textual changes made.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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
✨ 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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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)

7-7: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Fix additional typo in module documentation.

While fixing the "occured" → "occurred" typo, there's another spelling error on this line: "occurr" should be "occur" (one 'r').

📝 Proposed fix
-/// A unified error enum that contains several errors that might occurr during
+/// A unified error enum that contains several errors that might occur during
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/error.rs` at line 7, Fix the remaining typo in the module-level
documentation in src/error.rs: change "occurr" to "occur" in the doc comment
that begins "A unified error enum that contains several errors that might...".
This is the docstring above the Error enum; update that comment to read "occur"
(single 'r') so the sentence reads correctly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@src/error.rs`:
- Line 7: Fix the remaining typo in the module-level documentation in
src/error.rs: change "occurr" to "occur" in the doc comment that begins "A
unified error enum that contains several errors that might...". This is the
docstring above the Error enum; update that comment to read "occur" (single 'r')
so the sentence reads correctly.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: c3d5c97c-bf67-4f22-861b-f29da4af3269

📥 Commits

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

📒 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