Skip to content

fix: typos and grammar errors in messages and source comments#1547

Open
ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
ZLeventer:fix/typos-and-grammar
Open

fix: typos and grammar errors in messages and source comments#1547
ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
ZLeventer:fix/typos-and-grammar

Conversation

@ZLeventer
Copy link
Copy Markdown

Summary

Fixes 11 typos and grammar errors in CLI help text and source comments:

Messages (user-visible CLI help text)

  • delete.source.md, deploy.metadata.md, deploy.metadata.validate.md: The RunRelevantTests paragraph says "computed for each class and triggers individually" — inconsistent with the RunSpecifiedTests paragraph just above it (and Salesforce documentation), which correctly uses the singular "trigger"
  • deploy.metadata.cancel.md: "most recent deploy operations was" → "operation was"
  • deploy.metadata.quick.md, deploy.metadata.validate.md: "production org take several hours" → "takes"
  • list.ignored.md: "foreceignored" → "forceignored"
  • retrieve.start.md: "The path relative you provided" → "The relative path you provided"

Source comments

  • output.ts:29: "Sorts file responds" → "Sorts file responses" (JSDoc)
  • coverage.ts:126: "total line for which...total uncovered line" → "total lines...total uncovered lines" (JSDoc)
  • deploy.ts:310: "If this's not resolve" → "If resolution fails" (garbled catch-block comment)

Test plan

  • All changes are in string literals and comments only — no logic changes
  • yarn build passes
  • Existing tests pass

- delete.source.md, deploy.metadata.md, deploy.metadata.validate.md: "class and triggers individually" → "class and trigger" — the RunRelevantTests paragraph had "triggers" while the RunSpecifiedTests paragraph just above it (and Salesforce documentation) uses singular "trigger"
- deploy.metadata.cancel.md: "most recent deploy operations was" → "operation was"
- deploy.metadata.quick.md, deploy.metadata.validate.md: "production org take several hours" → "takes"
- list.ignored.md: "foreceignored" → "forceignored"
- retrieve.start.md: "The path relative you provided" → "The relative path you provided"
- output.ts: "Sorts file responds" → "Sorts file responses"
- coverage.ts: "total line" → "total lines" (2x) in JSDoc
- deploy.ts: "If this's not resolve" → "If resolution fails" (garbled comment)
@ZLeventer ZLeventer requested a review from a team as a code owner April 24, 2026 14:27
@jshackell-sfdc
Copy link
Copy Markdown
Contributor

Thank you! Sheepishly, your CLI writer :)

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.

2 participants