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

Clarify that release dates are approximate #481

Merged
merged 2 commits into from
Jul 5, 2023

Conversation

jbw976
Copy link
Member

@jbw976 jbw976 commented Jul 3, 2023

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.

@netlify
Copy link

netlify bot commented Jul 3, 2023

Deploy Preview for crossplane ready!

Name Link
🔨 Latest commit 9044a05
🔍 Latest deploy log https://app.netlify.com/sites/crossplane/deploys/64a546e5ee67440008aee22d
😎 Deploy Preview https://deploy-preview-481--crossplane.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@jeanduplessis jeanduplessis left a 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   

@jbw976 jbw976 force-pushed the release-dates branch 3 times, most recently from a7037e0 to f2e7898 Compare July 5, 2023 10:29
@jbw976
Copy link
Member Author

jbw976 commented Jul 5, 2023

@jeanduplessis I fixed all Vale errors within the entire doc, and fixed all warnings within this new section:

✖ 0 errors, 55 warnings and 0 suggestions in 1 file.

Can you take another look now sir? 🙇

Copy link
Collaborator

@jeanduplessis jeanduplessis left a comment

Choose a reason for hiding this comment

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

Thanks @jbw976 LGTM 🚀

@jeanduplessis jeanduplessis merged commit 8a94475 into crossplane:master Jul 5, 2023
6 of 7 checks passed
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