Skip to content

docs: update README with maintainer names - #337

Merged
lance merged 2 commits into
cloudevents:mainfrom
lance:update-readme
Sep 9, 2020
Merged

lance merged 2 commits into
cloudevents:mainfrom
lance:update-readme

Conversation

@lance

@lance lance commented Sep 3, 2020

Copy link
Copy Markdown
Member

Proposed Changes

This commit changes the README to include all currently active
contributors to and the originator of the repository.

Signed-off-by: Lance Ball lball@redhat.com

This commit changes the README to include all currently active
contributors to and the originator of the repository.

Signed-off-by: Lance Ball <lball@redhat.com>
@lance lance added the module/docs Module documentation changes label Sep 3, 2020
@lance
lance requested a review from a team September 3, 2020 20:45

@grant grant left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • I'm not sure if Fabio has maintained this repo in the last year. Ping him?
  • @lance your name should probably be first given you're most active and probably on Slack more

Otherwise looks good

@lance

lance commented Sep 3, 2020

Copy link
Copy Markdown
Member Author

@lance your name should probably be first given you're most active and probably on Slack more

I just put them in alphabetical order by first name. I'll reach out to Fabio

Signed-off-by: Lance Ball <lball@redhat.com>
@lance

lance commented Sep 9, 2020

Copy link
Copy Markdown
Member Author

@cloudevents/sdk-javascript-maintainers I have removed Fabio from the list of people to reach out to on Slack (tried pinging him on CNCF Slack with no response), and put maintainer names in alphabetical order by last name. Can I get a LGTM?

@lholmquist
lholmquist self-requested a review September 9, 2020 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module/docs Module documentation changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants