Skip to content

fix: typos, grammar errors, and incorrect import resume stage title#1427

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

fix: typos, grammar errors, and incorrect import resume stage title#1427
ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
ZLeventer:fix/typos-grammar-and-bugs

Conversation

@ZLeventer
Copy link
Copy Markdown

Summary

Fixes 11 typos, grammar errors, and minor bugs found across messages and source files:

Messages (CLI help text)

  • record.update.md: "specifiyng" → "specifying"
  • tree.import.md: "more then 200" → "more than 200"
  • bulk.status.md + data.resume.md: "a specific batches" → "a specific batch"
  • importApi.md: removed trailing backtick artifact from error message string
  • bulk.upsert.md: added missing <%= command.id %> template tag in first example (currently renders without the command name)

Source

  • bulkIngest.ts: removed stray ".1" suffix in deprecation JSDoc comment
  • bulkUtils.ts: renamed local variable columDelimitercolumnDelimiter (typo, missing "n")
  • update/record.ts: "smart of enough" → "smart enough" in comment
  • importFiles.ts: fixed unclosed backtick in JSDoc (--plan was missing closing backtick)

Bug fix

  • import/resume.ts: changed stageTitle from 'Updating data' to 'Importing data' — every other resume command matches its bulk counterpart (delete/resume → "Deleting data", upsert/resume → "Upserting data", update/resume → "Updating data"), but import/resume incorrectly showed "Updating data" instead of "Importing data"

Test plan

  • All changes are string/comment-only except columDelimiter rename (local variable, 3 occurrences in one function) and stageTitle fix
  • yarn build passes
  • Existing tests pass

- record.update.md: "specifiyng" → "specifying"
- tree.import.md: "more then" → "more than"
- bulk.status.md, data.resume.md: "a specific batches" → "a specific batch"
- importApi.md: remove trailing backtick artifact from error message
- bulk.upsert.md: add missing <%= command.id %> in first example
- bulkIngest.ts: remove stray ".1" in deprecation comment
- bulkUtils.ts: rename `columDelimiter` → `columnDelimiter`
- update/record.ts: "smart of enough" → "smart enough"
- import/resume.ts: fix stageTitle from "Updating data" to "Importing data"
  (every other resume command matches its bulk counterpart)
- importFiles.ts: fix unclosed backtick in JSDoc comment
@ZLeventer ZLeventer requested a review from a team as a code owner April 24, 2026 14:18
Copy link
Copy Markdown
Contributor

@jshackell-sfdc jshackell-sfdc left a comment

Choose a reason for hiding this comment

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

NOTE that this is for only the markdown file changes. Thank you!

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