-
Notifications
You must be signed in to change notification settings - Fork 119
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
Introduce inclusive naming check and update existing language #595
Conversation
I need to update this as the |
b1b9c75
to
b790537
Compare
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.
LGTM
b790537
to
6065b56
Compare
Updated the PR to have a reusable workflow based on feedback in slack: https://kubernetes.slack.com/archives/CH8KCCKA5/p1668621129532309 |
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.
We should address my comment before merging this PR, but after that I think we should be good to merge it
We are following the Inclusive Naming Initiative's guidance as that is what the CNCF supports.
6065b56
to
8a706cd
Compare
We are following the Inclusive Naming Initiative's guidance as that is what the CNCF supports.