From 6faaa9c88283f05ffa95240da10540a84e7b7e94 Mon Sep 17 00:00:00 2001 From: Zach Leventer Date: Fri, 24 Apr 2026 10:26:25 -0400 Subject: [PATCH] fix: typos and grammar errors in messages and source comments MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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) --- messages/delete.source.md | 2 +- messages/deploy.metadata.cancel.md | 2 +- messages/deploy.metadata.md | 2 +- messages/deploy.metadata.quick.md | 2 +- messages/deploy.metadata.validate.md | 4 ++-- messages/list.ignored.md | 2 +- messages/retrieve.start.md | 2 +- src/utils/coverage.ts | 2 +- src/utils/deploy.ts | 2 +- src/utils/output.ts | 2 +- 10 files changed, 11 insertions(+), 11 deletions(-) diff --git a/messages/delete.source.md b/messages/delete.source.md index eb689bf06..89f6b3945 100644 --- a/messages/delete.source.md +++ b/messages/delete.source.md @@ -106,7 +106,7 @@ Valid values are: - RunAllTestsInOrg — All tests in your org are run, including tests of managed packages. -- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and triggers individually and is different than the overall coverage percentage. +- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and trigger individually and is different than the overall coverage percentage. If you don’t specify a test level, the default behavior depends on the contents of your deployment package and target org. For more information, see "Running Tests in a Deployment" (https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_deploy_running_tests.htm) in the "Metadata API Developer Guide". diff --git a/messages/deploy.metadata.cancel.md b/messages/deploy.metadata.cancel.md index 5d6979270..9d4e49880 100644 --- a/messages/deploy.metadata.cancel.md +++ b/messages/deploy.metadata.cancel.md @@ -39,7 +39,7 @@ Use the job ID of the most recent deploy operation. # flags.use-most-recent.description -For performance reasons, this flag uses job IDs for deploy operations that started only in the past 3 days or less. If your most recent deploy operations was more than 3 days ago, this flag won't find a job ID. +For performance reasons, this flag uses job IDs for deploy operations that started only in the past 3 days or less. If your most recent deploy operation was more than 3 days ago, this flag won't find a job ID. # flags.wait.summary diff --git a/messages/deploy.metadata.md b/messages/deploy.metadata.md index cee72524a..e457183ec 100644 --- a/messages/deploy.metadata.md +++ b/messages/deploy.metadata.md @@ -104,7 +104,7 @@ Valid values are: - RunAllTestsInOrg — All tests in your org are run, including tests of managed packages. -- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and triggers individually and is different than the overall coverage percentage. +- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and trigger individually and is different than the overall coverage percentage. If you don’t specify a test level, the default behavior depends on the contents of your deployment package and target org. For more information, see "Running Tests in a Deployment" (https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_deploy_running_tests.htm) in the "Metadata API Developer Guide". diff --git a/messages/deploy.metadata.quick.md b/messages/deploy.metadata.quick.md index e6c5f7195..2a6622ed4 100644 --- a/messages/deploy.metadata.quick.md +++ b/messages/deploy.metadata.quick.md @@ -6,7 +6,7 @@ Quickly deploy a validated deployment to an org. Before you run this command, first create a validated deployment with the "<%= config.bin %> project deploy validate" command, which returns a job ID. Validated deployments haven't been deployed to the org yet; you deploy them with this command. Either pass the job ID to this command or use the --use-most-recent flag to use the job ID of the most recently validated deployment. For the quick deploy to succeed, the associated validated deployment must also have succeeded. -Executing this quick deploy command takes less time than a standard deploy because it skips running Apex tests. These tests were previously run as part of the validation. Validating first and then running a quick deploy is useful if the deployment to your production org take several hours and you don’t want to risk a failed deploy. +Executing this quick deploy command takes less time than a standard deploy because it skips running Apex tests. These tests were previously run as part of the validation. Validating first and then running a quick deploy is useful if the deployment to your production org takes several hours and you don’t want to risk a failed deploy. This command doesn't support source-tracking. The source you deploy overwrites the corresponding metadata in your org. This command doesn’t attempt to merge your source with the versions in your org. diff --git a/messages/deploy.metadata.validate.md b/messages/deploy.metadata.validate.md index 506ccfa7b..c231f9957 100644 --- a/messages/deploy.metadata.validate.md +++ b/messages/deploy.metadata.validate.md @@ -4,7 +4,7 @@ Validate a metadata deployment without actually executing it. # description -Use this command to verify whether a deployment will succeed without actually deploying the metadata to your org. This command is similar to "<%= config.bin %> project deploy start", except you're required to run Apex tests, and the command returns a job ID rather than executing the deployment. If the validation succeeds, then you pass this job ID to the "<%= config.bin %> project deploy quick" command to actually deploy the metadata. This quick deploy takes less time because it skips running Apex tests. The job ID is valid for 10 days from when you started the validation. Validating first is useful if the deployment to your production org take several hours and you don’t want to risk a failed deploy. +Use this command to verify whether a deployment will succeed without actually deploying the metadata to your org. This command is similar to "<%= config.bin %> project deploy start", except you're required to run Apex tests, and the command returns a job ID rather than executing the deployment. If the validation succeeds, then you pass this job ID to the "<%= config.bin %> project deploy quick" command to actually deploy the metadata. This quick deploy takes less time because it skips running Apex tests. The job ID is valid for 10 days from when you started the validation. Validating first is useful if the deployment to your production org takes several hours and you don’t want to risk a failed deploy. You must run this command from within a project. @@ -52,7 +52,7 @@ Valid values are: - RunAllTestsInOrg — All tests in your org are run, including tests of managed packages. -- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and triggers individually and is different than the overall coverage percentage. +- RunRelevantTests (Beta) — Runs only tests that are relevant to the files being deployed. Salesforce automatically identifies the relevant tests based on an analysis of the deployment payload and the payload dependencies. For fine-grained control, you can also annotate test classes so that they always run in certain conditions. See "@IsTest Annotation" in the "Apex Developer Guide" (https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_classes_annotation_isTest.htm). Each class and trigger in the deployment package must be covered by the executed tests for a minimum of 75% code coverage. This coverage is computed for each class and trigger individually and is different than the overall coverage percentage. If you don’t specify a test level, the default behavior depends on the contents of your deployment package and target org. For more information, see "Running Tests in a Deployment" (https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_deploy_running_tests.htm) in the "Metadata API Developer Guide". diff --git a/messages/list.ignored.md b/messages/list.ignored.md index 932bd4cd4..67569e576 100644 --- a/messages/list.ignored.md +++ b/messages/list.ignored.md @@ -22,7 +22,7 @@ When deploying or retrieving metadata between your local project and an org, you # flags.source-dir.summary -File or directory of files that the command checks for foreceignored files. +File or directory of files that the command checks for forceignored files. # invalidSourceDir diff --git a/messages/retrieve.start.md b/messages/retrieve.start.md index f03725cb1..3fff7aa7e 100644 --- a/messages/retrieve.start.md +++ b/messages/retrieve.start.md @@ -207,4 +207,4 @@ Specifically, to use this command with a production org, scratch org created wit # outputDirOutsideProject -The output directory must be inside the current project. The path relative you provided %s is outside the project root. +The output directory must be inside the current project. The relative path you provided %s is outside the project root. diff --git a/src/utils/coverage.ts b/src/utils/coverage.ts index 0969f1b64..7efc6b822 100644 --- a/src/utils/coverage.ts +++ b/src/utils/coverage.ts @@ -123,7 +123,7 @@ export const getCoveragePct = (cov: CodeCoverage): number => { return lineCount > 0 ? coverageDecimal * 100 : 100; }; -/** returns the number of total line for which coverage should apply, and the total uncovered line */ +/** returns the number of total lines for which coverage should apply, and the total uncovered lines */ export const getCoverageNumbers = (cov: CodeCoverage): [lineCount: number, uncoveredLineCount: number] => [ parseInt(cov.numLocations, 10), parseInt(cov.numLocationsNotCovered, 10), diff --git a/src/utils/deploy.ts b/src/utils/deploy.ts index 8fdd80e4f..b264f769b 100644 --- a/src/utils/deploy.ts +++ b/src/utils/deploy.ts @@ -307,7 +307,7 @@ export async function buildPreDestructiveFileResponses( preDestructiveComponents.length = 0; preDestructiveComponents.push(...resolvedComponents); } catch (error) { - // If this's not resolve, try to resolve with registry only + // If resolution fails, try to resolve with registry only } for (const component of preDestructiveComponents) { diff --git a/src/utils/output.ts b/src/utils/output.ts index b11d060d7..5434178b0 100644 --- a/src/utils/output.ts +++ b/src/utils/output.ts @@ -26,7 +26,7 @@ export function tableHeader(message: string): string { export const makePathRelative = (fr: T): T => fr.filePath ? { ...fr, filePath: path.relative(process.cwd(), fr.filePath) } : fr; /** - * Sorts file responds by type, then by filePath, then by fullName + * Sorts file responses by type, then by filePath, then by fullName */ export const fileResponseSortFn = (i: FileResponse, j: FileResponse): number => { if (i.type === j.type && i.filePath && j.filePath) {