Skip to content

CSpell Action - #75

Merged
Skptak merged 13 commits into
FreeRTOS:mainfrom
Skptak:cSpellChanges
Sep 5, 2023
Merged

Skptak merged 13 commits into
FreeRTOS:mainfrom
Skptak:cSpellChanges

Conversation

@Skptak

@Skptak Skptak commented Aug 15, 2023

Copy link
Copy Markdown
Contributor

Splitting up #68 into more manageable single PRs containing just the changes to the individual actions
Will not be doing the YML files one by one, so please reference the #68 actions for proof of working

This exists more or less just to push the cpsell config to users for use with VSCode

Relevant Log can be found here

Comment thread spellings/cspell.config.yaml Outdated
Comment thread spellings/getFiles Outdated
@kstribrnAmzn

Copy link
Copy Markdown
Member

Does this PR force any changes to our existing lexicons setup in our repository? Or is the behavior setup to be identical to the old spell checker and it's dictionaries?

Comment thread spellings/getFiles Outdated
Comment thread spellings/action.yml
paulbartell
paulbartell previously approved these changes Aug 21, 2023
@Skptak

Skptak commented Aug 22, 2023

Copy link
Copy Markdown
Contributor Author

Does this PR force any changes to our existing lexicons setup in our repository? Or is the behavior setup to be identical to the old spell checker and it's dictionaries?

The idea is to provide a cspell configuration that can then be tied into a plugin for visual studio code plugin that uses the same format
Where we then use the same dictionary for that spell checker as we do for the rust-spell-check that gets run on PRs as it's faster

…iles directly, update test to account for it, move the .cSpellWords.txt file to live in .github
A-Zaba
A-Zaba previously approved these changes Aug 31, 2023
Comment thread .github/workflows/test.yml
Comment thread .github/workflows/test.yml
A-Zaba
A-Zaba previously approved these changes Sep 5, 2023
@Skptak
Skptak merged commit a418b2b into FreeRTOS:main Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants