Skip to content
This repository was archived by the owner on Feb 14, 2023. It is now read-only.

Conversation

@kianmeng
Copy link

Found via codespell -S target,*.txt -L crate

Found via `codespell -S target,*.txt -L crate`
Copy link

@rillian rillian left a comment

Choose a reason for hiding this comment

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

The "occurred" fixes are correct and complete. "supercede" is a style question, but would be fine to take.

However, this is a huge patch for a handful of spelling corrections. It would be much easier to review if you split the cargo fmt changes into a separate commit. Would you mind doing that?

}

/// Processes a request and returns response or an occured error.
/// Processes a request and returns response or an occurred error.
Copy link

Choose a reason for hiding this comment

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

Typo is fixed here.

@kianmeng
Copy link
Author

kianmeng commented Feb 3, 2023

@rillian Separate PR for cargo fmt created, #147.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants