Fix typos in code comments#9363
Conversation
|
I'm starting a first review of this pull request. You can follow along in the session on Warp. I approved this pull request. No matching stakeholder was found for the changed files, so no human reviewers were requested. Comment I approved this pull request and requested human review from: @zachbai, @kevinyang372, @vkodithala. Comment Powered by Oz |
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR fixes spelling typos in comments and Rust doc comments across AI, notebook, pane, telemetry, terminal, workspace, editor, server-client, and shell files. The diff does not change executable code.
Concerns
- No correctness, security, error handling, or performance concerns found in the changed lines.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
There was a problem hiding this comment.
Overview
This PR corrects spelling typos in Rust comments and doc comments across app and crate files; no functional code or configuration changes are present.
Concerns
- One changed doc comment in
app/src/server/telemetry/events.rsstill reads incorrectly asif the some part. - No security findings from the supplemental security pass.
Verdict
Found: 0 critical, 0 important, 1 suggestion
Approve with nits
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| /// The server-generated output ID for the output in this block. | ||
| /// | ||
| /// This is only populated if the some part of the response was succesfully received. | ||
| /// This is only populated if the some part of the response was successfully received. |
There was a problem hiding this comment.
💡 [SUGGESTION] This doc comment still reads incorrectly; drop the before some.
| /// This is only populated if the some part of the response was successfully received. | |
| /// This is only populated if some part of the response was successfully received. |
captainsafia
added
the
external-contributor
Warp Dev Github Integration
Fixes 18 spelling typos across 16 code-comment locations. All changes are inside `//` or `///` comments; no functional code is touched. Categories: - `concious` -> `conscious`, `Wether` -> `Whether`, `psuedo` -> `pseudo`, `occassion` -> `occasion`, `calcuates` -> `calculates` (single-word slips) - `succesful` / `succesfully` -> `successful` / `successfully` (x3) - `recieve` / `recieved` / `recieves` -> `receive` / `received` / `receives` (x4) - `occured` -> `occurred` (x2) - `seperate`, `seperator`, `indvidual`, `accross`, `refering` -> standard spellings Skipped intentionally: - `crates/ai/src/diff_validation/mod_test.rs:13` - the typo there is inside a verbatim GNU/Linux copypasta string used as test input. - `resources/bundled/mcp_skills/figma/...` - vendor documentation. Comment-only changes; no test impact. - [ ] Warp Agent Mode - This PR was created via Warp's AI Agent Mode Co-authored-by: BennyWaitWhat <[email protected]>
3 participants
Description
Fixes 18 spelling typos across 16 code-comment locations. All changes are inside
//or///comments; no functional code is touched. Categories:concious->conscious,Wether->Whether,psuedo->pseudo,occassion->occasion,calcuates->calculates(single-word slips)succesful/succesfully->successful/successfully(x3)recieve/recieved/recieves->receive/received/receives(x4)occured->occurred(x2)seperate,seperator,indvidual,accross,refering-> standard spellingsSkipped intentionally:
crates/ai/src/diff_validation/mod_test.rs:13- the typo there is inside a verbatim GNU/Linux copypasta string used as test input.resources/bundled/mcp_skills/figma/...- vendor documentation.Testing
Comment-only changes; no test impact.
Agent Mode