fix: typos, grammar errors, and incorrect import resume stage title#1427
Open
ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
Open
fix: typos, grammar errors, and incorrect import resume stage title#1427ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
ZLeventer wants to merge 1 commit intosalesforcecli:mainfrom
Conversation
- 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
jshackell-sfdc
approved these changes
Apr 24, 2026
Contributor
jshackell-sfdc
left a comment
There was a problem hiding this comment.
NOTE that this is for only the markdown file changes. Thank you!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 stringbulk.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 commentbulkUtils.ts: renamed local variablecolumDelimiter→columnDelimiter(typo, missing "n")update/record.ts: "smart of enough" → "smart enough" in commentimportFiles.ts: fixed unclosed backtick in JSDoc (--planwas missing closing backtick)Bug fix
import/resume.ts: changedstageTitlefrom'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"), butimport/resumeincorrectly showed "Updating data" instead of "Importing data"Test plan
columDelimiterrename (local variable, 3 occurrences in one function) andstageTitlefixyarn buildpasses