Skip to content

Conversation

ValentinGerlach
Copy link
Member

@ValentinGerlach ValentinGerlach commented Aug 20, 2025

What this PR does / why we need it:

This PR adds prose linting using Vale. The configuration and rules can be adjusted in the future when we have more experience with the tool. For now, it is configured to follow Google's developer documentation style guide.

All warnings have been resolved. There are just some suggestions that can be ignored for now.

@ValentinGerlach ValentinGerlach marked this pull request as ready for review August 26, 2025 09:57
Copy link

@Lasserich Lasserich left a comment

Choose a reason for hiding this comment

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

minor remarks, but lgtm mostly!

name: replace
swap:
are not: aren't
cannot: can't

Choose a reason for hiding this comment

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

Suggested change
cannot: can't
can not: can't

@@ -0,0 +1,30 @@
extends: substitution
message: "Use '%s' instead of '%s'."
link: 'https://developers.google.com/style/contractions'

Choose a reason for hiding this comment

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

This one says to only contract "dismisses" with "not". I am fine with keeping the extra work tho!

swap:
(?:alumna|alumnus): graduate
(?:alumnae|alumni): graduates
air(?:m[ae]n|wom[ae]n): pilot(s)

Choose a reason for hiding this comment

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

This is a thing? Haha never heard of before.

Choose a reason for hiding this comment

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

Where does this list come from, do we maintain it?

@@ -1,3 +1,3 @@
# Managed Control Planes (MCPs)
# Managed control planes (MCPs)

Choose a reason for hiding this comment

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

This should remain "Managed Control Planes" imo.

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