-
Notifications
You must be signed in to change notification settings - Fork 772
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
update README with SIG-APPS and Champion #201
Conversation
Incubation of `kompose` into the Kubernetes project will be discussed in the [API Machinery SIG](https://github.com/kubernetes/community) | ||
Incubation of `kompose` into the Kubernetes project is sheppered in the [SIG-APPS](https://github.com/kubernetes/community). | ||
|
||
@ericchiang is acting as Champion for incubation |
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.
can we make his name as link?
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.
@surajssd fixed
3e22564
to
1678ebb
Compare
Incubation of `kompose` into the Kubernetes project will be discussed in the [API Machinery SIG](https://github.com/kubernetes/community) | ||
Incubation of `kompose` into the Kubernetes project is shepherd in the [SIG-APPS](https://github.com/kubernetes/community). | ||
|
||
[@ericchiang](https://github.com/ericchiang) is acting as Champion for incubation |
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.
missing period after "incubation". perhaps add link?
(incupation)[https://github.com/kubernetes/community/blob/master/incubator.md]
since i had to google what "Champion" was and got linked to the incubation doc on kubernetes community
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, fixed.
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!
I am gonna make it merge-able by adding approved review. @sebgoa has the whole idea on the incubating progress. |
ok merging |
some clean up for incubation