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

Do a pass on spelling and markdown linting in markdown only #216

Merged
merged 4 commits into from
Sep 4, 2023

Conversation

michael-hawker
Copy link
Member

Caught one issue with spelling in a parameter name as it was in the sample, but didn't explicitly spell check code or sample code...

We'll probably want to do a more complete pass on the repo itself, as well as setup https://github.com/marketplace/actions/check-spelling

Caught one issue with spelling in a parameter name as it was in the sample,
but didn't explicitly spell check code or sample code...
@michael-hawker michael-hawker added the documentation Improvements or additions to documentation label Sep 2, 2023
@michael-hawker
Copy link
Member Author

@Arlodotexe you may just want to setup a separate job like the Xaml-Style-Check that runs in the build.yml as a separate branch/PR on Tuesday and see what it finds (assuming it scans the whole repo vs. what's changed).

We'll want to identify any spelling mistakes in APIs that we may want to fix before 8.0 final vs. bringing them forward from 7.x.

components/Segmented/samples/Segmented.md Outdated Show resolved Hide resolved
@michael-hawker
Copy link
Member Author

Failed during test pass (probably usual issue)

@niels9001 niels9001 enabled auto-merge (squash) September 4, 2023 10:03
@niels9001 niels9001 merged commit c810a39 into main Sep 4, 2023
9 checks passed
@delete-merged-branch delete-merged-branch bot deleted the llama/typo-pass branch September 4, 2023 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants