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 vale to PR workflow #1780

Closed
wants to merge 1 commit into from
Closed

Add vale to PR workflow #1780

wants to merge 1 commit into from

Conversation

kaylareopelle
Copy link
Contributor

The docs-website uses this linter in its CI.
Since its introduction, we've needed to update change log entries after the fact.
Let's add it to our PR workflow, in hopes that we can preemptively address its feedback.

https://vale.sh/

The docs-website uses this linter in its CI.
Since its introductino, we've needed to update change log entries
after the fact.
Let's add it to our PR workflow, in hopes that we can preemptively
address its feedback.
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2023

SimpleCov Report

Coverage Threshold
Line 93.22% 93%
Branch 84.83% 84%

@kaylareopelle
Copy link
Contributor Author

There's a lot of repo-specific config files that would need to stay in sync with the docs-website. I don't think this is ready to be added at this time. I opened an issue on the docs-website requesting a custom GitHub action with the styles, ini file, etc. for distribution to NR teams.

newrelic/docs-website#11655

@kaylareopelle kaylareopelle deleted the add-vale-linter branch February 21, 2023 17:37
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.

1 participant