-
Notifications
You must be signed in to change notification settings - Fork 110
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
Clarify that release dates are approximate #481
Conversation
✅ Deploy Preview for crossplane ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jbw976 would you mind having a look at the Vale errors for this new section
86:4 warning 'Release Dates' should use Google.Headings
sentence-style capitalization.
88:12 warning 'is released' may be passive write-good.Passive
voice. Use active voice if you
can.
88:29 warning 'quarterly' is a weasel word! write-good.Weasel
89:1 warning Avoid using future tense: gitlab.FutureTense
"will happen". Use present
tense instead.
90:43 warning Remove this word from the gitlab.Wordy
sentence: "Please".
91:26 warning 'A number of' is too wordy. write-good.TooWordy
Signed-off-by: Jared Watts <[email protected]>
a7037e0
to
f2e7898
Compare
…es section Signed-off-by: Jared Watts <[email protected]>
@jeanduplessis I fixed all Vale errors within the entire doc, and fixed all warnings within this new section:
Can you take another look now sir? 🙇 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jbw976 LGTM 🚀
As a follow-up here in the docs from crossplane/crossplane#4211, this PR clarifies that the release dates are approximate and provides reasoning why.