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

Changes per TAG Contrib Strategy review #1224

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Conversation

duglin
Copy link
Collaborator

@duglin duglin commented Sep 11, 2023

Misc edits per the TAG Contrib Strategy Review: cncf/tag-contributor-strategy#509

README.md Outdated
@@ -139,6 +145,9 @@ For SDK related comments and questions:
- Archives are at: https://lists.cncf.io/g/cncf-cloudevents-sdk/topics
- Slack: #cloudeventssdk on [CNCF's Slack workspace](http://slack.cncf.io/)

For SDK specific communications, please see the main README in each
SDK's github repo - see the list of [SDKs above](#sdks).
Copy link
Contributor

Choose a reason for hiding this comment

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

Possibly change the link text to "list of SDKs" rather than "SDKs above"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@@ -5,6 +5,16 @@
This document describes the governance process under which the CloudEvents
project will manage this repository.

For easy reference, additional documentation related to how this project
(and it's subprojects) operate are listed below:
Copy link
Contributor

Choose a reason for hiding this comment

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

it's => its

Copy link
Contributor

Choose a reason for hiding this comment

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

(And I'd probably remove the parentheses - otherwise if you ignore the parenthetical part, you get a singular/plural mismatch.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@duglin
Copy link
Collaborator Author

duglin commented Sep 14, 2023

As always, thanks for the review @jskeet

Approved on the 9/14 call.

@duglin duglin merged commit 7f0d456 into cloudevents:main Sep 14, 2023
@duglin duglin deleted the tag-review branch September 14, 2023 17:03
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