Skip to content

Add output for failed spellcheck runs#641

Merged
AaronCrawfis merged 8 commits into
v0.22from
aacrawfi/spellcheck-output
Jul 6, 2023
Merged

Add output for failed spellcheck runs#641
AaronCrawfis merged 8 commits into
v0.22from
aacrawfi/spellcheck-output

Conversation

@AaronCrawfis
Copy link
Copy Markdown

Adds steps so failed spellcheck creates a new comment and workflow output

@AaronCrawfis AaronCrawfis requested a review from a team as a code owner July 6, 2023 16:57
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

@radius-project radius-project deleted a comment from github-actions Bot Jul 6, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

2 similar comments
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

branches:
- v*.*
- edge
paths:
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Removing these so spellcheck always runs. Needed since we require spellcheck to pass before merging

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 6, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-641.westus2.azurestaticapps.net
Password: Radiu$1234

Copy link
Copy Markdown
Contributor

@willtsai willtsai left a comment

Choose a reason for hiding this comment

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

@AaronCrawfis AaronCrawfis merged commit dcd1c2d into v0.22 Jul 6, 2023
@AaronCrawfis AaronCrawfis deleted the aacrawfi/spellcheck-output branch July 6, 2023 23:03
willtsai added a commit that referenced this pull request Jul 7, 2023
* Create .gitattributes

* Add dropdown arrows to sections (#639)

* Add output for failed spellcheck runs (#641)

* Add output steps

* Add misspelled word

* Switch comment provider

* Clear comment on success

* Rename job

* Add concurrency control

* Fix spelling issue

* Disable section pages (#644)

* Disable section page nav

* Update section pages

* Apply suggestions from code review

Co-authored-by: Will <28876888+willtsai@users.noreply.github.com>

* Remove _ for leaf pages

---------

Co-authored-by: Will <28876888+willtsai@users.noreply.github.com>

* Update sidebar GitHub links and taxonomies (#638)

* Update GitHub issues links

* Only show tags/categories on home page

* v0.22 to edge upmerge

Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>

* v0.22 to edge upmerge

Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>

---------

Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>
Co-authored-by: Aaron Crawfis <Aaron.Crawfis@microsoft.com>
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.

2 participants