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

feat: add Maintainers Growth Working Group #1143

Merged
merged 27 commits into from
May 8, 2024

Conversation

smoya
Copy link
Member

@smoya smoya commented Apr 2, 2024

Description

This PR materializes the creation of the Maintainers Growth Working Group, as discussed in https://github.com/orgs/asyncapi/discussions/1094.

It replaces #1090.

Additionally, it introduces a new format on the WORKING_GROUPS.yaml file so we can expand metadata about members, such as slack user id, etc as per https://github.com/asyncapi/community/pull/1090/files#r1537111382.

WORKING_GROUPS.yaml Outdated Show resolved Hide resolved
WORKING_GROUPS.yaml Outdated Show resolved Hide resolved
WORKING_GROUPS.yaml Outdated Show resolved Hide resolved
WORKING_GROUPS.yaml Outdated Show resolved Hide resolved
@quetzalliwrites quetzalliwrites changed the title feat: add Community Growth Working Group feat: add Maintainers Growth Working Group Apr 15, 2024
@smoya smoya requested review from fmvilas and Shurtu-gal April 16, 2024 17:50
- name: Maintainers Growth
description: The Maintainers Working Group aims to help grow the number of maintainers at AsyncAPI.
chairperson:
name: Sergio Moya
Copy link
Member Author

Choose a reason for hiding this comment

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

Added myself, but I still believe @fmvilas should be the one. Discussion happening in https://github.com/orgs/asyncapi/discussions/1094#discussioncomment-9132744.

But we can move forward and merge this, we can change later if applies.

@smoya
Copy link
Member Author

smoya commented Apr 16, 2024

@alequetzalli @derberg @thulieblack @fmvilas @Mayaleeeee @AceTheCreator this PR is now ready to review!

@smoya
Copy link
Member Author

smoya commented Apr 19, 2024

/help

@smoya smoya requested a review from fmvilas April 19, 2024 08:54
@asyncapi-bot
Copy link
Contributor

Hello, @smoya! 👋🏼

    I'm 🧞🧞🧞 Genie 🧞🧞🧞 from the magic lamp. Looks like somebody needs a hand!

    At the moment the following comments are supported in pull requests:

    - `/please-take-a-look` or `/ptal` - This comment will add a comment to the PR asking for attention from the reviewrs who have not reviewed the PR yet.
    - `/ready-to-merge` or `/rtm` - This comment will trigger automerge of PR in case all required checks are green, approvals in place and do-not-merge label is not added
    - `/do-not-merge` or `/dnm` - This comment will block automerging even if all conditions are met and ready-to-merge label is added
    - `/autoupdate` or `/au` - This comment will add `autoupdate` label to the PR and keeps your PR up-to-date to the target branch's future changes. Unless there is a merge conflict or it is a draft PR. (Currently only works for upstream branches.)
    - `/update` or `/u` - This comment will update the PR with the latest changes from the target branch. Unless there is a merge conflict or it is a draft PR. NOTE: this only updates the PR once, so if you need to update again, you need to call the command again.

@smoya
Copy link
Member Author

smoya commented Apr 19, 2024

/ptal

@asyncapi-bot
Copy link
Contributor

Please take a look at this PR. Thanks! 👋

github: smoya
slack: UN22ZTLHG
- name: Alejandra Quetzalli
Copy link
Member Author

Choose a reason for hiding this comment

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

@alequetzalli 👋

Copy link
Member

Choose a reason for hiding this comment

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

done!

quetzalliwrites
quetzalliwrites previously approved these changes May 7, 2024
@smoya
Copy link
Member Author

smoya commented May 7, 2024

/ptal

@asyncapi-bot
Copy link
Contributor

@thulieblack Please take a look at this PR. Thanks! 👋

Copy link
Member

@thulieblack thulieblack left a comment

Choose a reason for hiding this comment

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

🫡

@smoya
Copy link
Member Author

smoya commented May 8, 2024

@fmvilas can you approve? You requested changes that were applied.

@smoya
Copy link
Member Author

smoya commented May 8, 2024

/rtm

@asyncapi-bot asyncapi-bot merged commit 7f6a292 into asyncapi:master May 8, 2024
8 checks passed
@smoya smoya deleted the feat/cgwg branch May 8, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

8 participants