docs(spelling): standardize spelling for consistency#120
Open
OmarElzero wants to merge 1 commit intoaccordproject:mainfrom
Open
docs(spelling): standardize spelling for consistency#120OmarElzero wants to merge 1 commit intoaccordproject:mainfrom
OmarElzero wants to merge 1 commit intoaccordproject:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR aims to standardize American English spelling across documentation/template fixtures to improve consistency, as identified by cspell.
Changes:
- Replaces “colours” → “colors” in template fixtures and updates corresponding snapshot output.
- Replaces “Defence” → “Defense” in an insurance template fixture and updates corresponding snapshot output.
- Introduces a structural change in
optional-nested/template.mdaround the optionalagerendering (beyond spelling).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| test/templates/good/playground/template.md | Spelling update for “colors” in the playground template fixture. |
| test/templates/good/optional-nested/template.md | Spelling update plus a behavioral change to how the age line is emitted (now wrapped in an #optional). |
| test/templates/good/insurance-sub-limits-liability/template.md | Spelling update for “Defense Costs” in the insurance sub-limits fixture. |
| test/snapshots/TemplateMarkInterpreter.test.ts.snap | Snapshot updates to match the template text/AST changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
5e8470e to
a6def2d
Compare
Fixes accordproject#119 Signed-off-by: OmarElzero <omarmohmed7659944@gmail.com>
a6def2d to
1b5a1fc
Compare
Author
|
Just a quick note to clarify the history here. This branch had some tangled commits from a previous PR by mistake. I've cleaned it up now, and this PR only contains the spelling standardization as intended. The outdated review comment can be disregarded. |
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.
Fixes #119
This PR standardizes the spelling of several words across the documentation and template files for better consistency.
Defencehas been changed toDefense.colourshas been changed tocolors.This was identified by running cspell across the repository's markdown files.