Skip to content

Fix more typos in comments and docs#9404

Open
BennyWaitWhat wants to merge 1 commit intowarpdotdev:masterfrom
BennyWaitWhat:bennywaitwhat/fix-more-comment-typos
Open

Fix more typos in comments and docs#9404
BennyWaitWhat wants to merge 1 commit intowarpdotdev:masterfrom
BennyWaitWhat:bennywaitwhat/fix-more-comment-typos

Conversation

@BennyWaitWhat
Copy link
Copy Markdown
Contributor

Description

A third batch of unambiguous spelling and grammar typos in comments and Markdown docs. 18 files, +26/-26 lines, 29 individual typo corrections. All changes are inside // or /// comments or Markdown text; no functional code is touched.

Categories:

  • propogate / propogates / propogation / propgate -> propagate / etc. (most common, 8+ instances across terminal/view.rs, tips/tip_view.rs, resource_center/main_page.rs, server/sync_queue.rs, integration/tests/common/mod.rs)
  • reponder -> responder (in tip_view.rs, main_page.rs, code_review/mod.rs)
  • accomodate -> accommodate, enviornment -> environment, intialize -> initialize, hierachy -> hierarchy, paramter -> parameter, delimeter -> delimiter (comment only; the WARP_SECRET_DELIMETER constant and string literal on the next line are intentionally left untouched - they're part of an on-disk data format)
  • appearence -> appearance, precicely -> precisely, handlign -> handling, implmentation -> implementation, throughought -> throughout, comands -> commands
  • local_command_executor.rs:176 had two typos in one line (then -> the, enviornment -> environment)
  • Article errors in doc comments: An unique reference -> A unique reference, an user input -> a user input (both nouns start with a /j/ sound and take "a")
  • Propogate -> Propagate

Skipped intentionally:

  • The WARP_SECRET_DELIMETER constant name and /warp-secret-delimeter/ string literal in external_secrets/mod.rs:22 (data format, separate refactor concern).
  • The delimeter parameter name and call sites in env_vars/mod.rs (code-identifier rename, also a separate refactor).
  • Test fixtures that contain verbatim quoted text.

Testing

Comment / docs only; no test impact.

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

@cla-bot cla-bot Bot added the cla-signed label Apr 29, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented Apr 29, 2026

@BennyWaitWhat

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I approved this pull request. No matching stakeholder was found for the changed files, so no human reviewers were requested.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This PR corrects spelling and grammar in comments and Markdown documentation only. The reviewed diff does not change executable logic, and the supplemental security pass found no security-impacting changes.

Concerns

  • One changed doc-comment line still has an adjacent grammar issue.

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

}

/// A [`SingletonEntity`] that the tracks events for the code review model throughought the app.
/// A [`SingletonEntity`] that the tracks events for the code review model throughout the app.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 [SUGGESTION] Remove the extra article while this doc comment is being cleaned up.

Suggested change
/// A [`SingletonEntity`] that the tracks events for the code review model throughout the app.
/// A [`SingletonEntity`] that tracks events for the code review model throughout the app.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant