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

Make vale work #3261

Merged
merged 33 commits into from
Sep 12, 2024
Merged

Make vale work #3261

merged 33 commits into from
Sep 12, 2024

Conversation

mooreds
Copy link
Contributor

@mooreds mooreds commented Sep 11, 2024

  • made vale fail on error
  • fixed a ton of misspellings or incorrect casing
  • wrote ignore clauses for some more cases
  • updated dev docs to reflect new vale work
  • fixed incorrect example apps count

@mooreds mooreds requested review from a team as code owners September 11, 2024 03:47
looks like we had a couple of issues
@lyleschemmerling
Copy link
Contributor

how were you testing the updates? Just letting the github action run on commit, or are you running this locally?

@mooreds
Copy link
Contributor Author

mooreds commented Sep 11, 2024

Running locally. Per devdocsreadme, you can run with vale --config .vale.ini astro/src/content or a more narrowly focused path. That is if you have installed vale locally, which I have. I think there's also a way to run it via docker if you don't want to install it.

@mooreds
Copy link
Contributor Author

mooreds commented Sep 11, 2024

I plan to ask @synedra to review it tomorrow morning first, then ask for your blessing @lyleschemmerling , because of the GH action change (which is minor).

@LucasPMorris
Copy link
Contributor

LucasPMorris commented Sep 11, 2024 via email

@mooreds
Copy link
Contributor Author

mooreds commented Sep 11, 2024

@LucasPMorris go back to your vacation! :)

Seriously, I didn't pick the reviewers; it happens automatically. It happens because of a change to codeowners ( d347d1a ).

I don't know what drove that change, wasn't involved.

@LucasPMorris
Copy link
Contributor

LucasPMorris commented Sep 11, 2024 via email

@mooreds mooreds mentioned this pull request Sep 11, 2024
Copy link
Contributor

@synedra synedra left a comment

Choose a reason for hiding this comment

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

It's clear and I'm excited to have a new tool to make our content more consistent!

@mooreds mooreds merged commit 2008149 into main Sep 12, 2024
3 checks passed
@mooreds mooreds deleted the mooreds/fix-vale-oversensitivity branch September 12, 2024 19:27
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.

4 participants