(maint) Add spell check listing to repository #855
Closed
Conversation
This file contains 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
AdmiringWorm
force-pushed
the
spell-checking
branch
from
d29c00b to
1f0069f
Compare
|
Converted this to draft as it seems some of our files are too long to be usable with the LanguageTool action (at least without change). |
gep13
reviewed
Oct 4, 2023
| @@ -85,7 +85,7 @@ You will also need to have at least one Group of computers already defined. | |||
| By default, if any Deployment Step fails, the overall Deployment is marked as Failed. | |||
| * `Only run other Deployment Steps if successful`. | |||
| Enabling this option will prevent subsequent Deployment Steps from starting if this step fails. | |||
| The overall Deployment will be marked as Failed, and subsequent steps will be Cancelled. | |||
| The overall Deployment will be marked as Failed, and subsequent steps will be Canceled. | |||
There was a problem hiding this comment.
We use Cancelled everywhere in the CCM application, so I don't think we can continue with this suggestion. I remember a discussion at the time about what should be used, as it wasn't immediately clear what was the right answer, and we settled on Cancelled.
AdmiringWorm
force-pushed
the
spell-checking
branch
from
1f0069f to
1699bae
Compare
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.
Description Of Changes
This PR adds two new workflows that uses misspell and LanguageTool
to run verification on the changed text in a PR, or the entire repository
on demand to verify that there are no spelling mistakes and that sentences
are correct.
Motivation and Context
We should strive to not have spelling mistakes when merging PR's into the repository.
Testing
Workflows tested through my own fork
Operating Systems Testing
N/A
Change Types Made
Change Checklist
Related Issue
N/A