Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add spellcheck and fix misspellings #523

Merged
merged 3 commits into from
Dec 7, 2022
Merged

Add spellcheck and fix misspellings #523

merged 3 commits into from
Dec 7, 2022

Conversation

sunker
Copy link
Contributor

@sunker sunker commented Dec 6, 2022

Spellchecks will soon be required in CI, so this PR adds a spellcheck script to package json. There were a lot of issues in this repo. I have fixed a few obvious problems, but feel free to take over this PR and take it further.

@sunker sunker requested a review from a team as a code owner December 6, 2022 13:55
@CLAassistant
Copy link

CLAassistant commented Dec 6, 2022

CLA assistant check
All committers have signed the CLA.

@sunker sunker changed the title add spellcheck script Add spellcheck and fix misspellings Dec 6, 2022
Copy link
Contributor

@aangelisc aangelisc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR @sunker!

@github-actions
Copy link

github-actions bot commented Dec 6, 2022

Backend code coverage report for PR #523

Plugin Main PR Difference
azuredx 33.8% 33.8% 0%

@github-actions
Copy link

github-actions bot commented Dec 6, 2022

Frontend code coverage report for PR #523

Plugin Main PR Difference
src 80.36% 81.86% 1.50%

Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@sunker sunker merged commit f4dd344 into main Dec 7, 2022
@sunker sunker deleted the add-spellcheck branch December 7, 2022 09:12
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.

None yet

4 participants