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

docs: add info about working groups #1075

Merged
merged 9 commits into from
Feb 27, 2024

Conversation

fmvilas
Copy link
Member

@fmvilas fmvilas commented Feb 22, 2024

Description
Adds info about how working groups work and an initial WORKING_GROUPS.yaml file with an example on how to define a working group.

Once this is merged. I'll start pinging people so they create individual PRs for the first working groups.

Related issue(s)
https://github.com/orgs/asyncapi/discussions/1037

@fmvilas
Copy link
Member Author

fmvilas commented Feb 22, 2024

Just drafted an initial version of this information. Please let me know if something should or shouldn't be required, or if something is missing.

# chairperson: Required. GitHub handle of the chairperson. Example: @Barbanio.
# members: # Required to have at least 2 members who are not the same as the chairperson.
# - @fmvilas # GitHub handle of the member.
# okrs_url: https://example.com/xyz # Required. Link to a GitHub project, issue, or any other tool where the Working Group explains their objectives.
Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure if this should be required. As per the rules described in the Markdown file, it's required but I'm happy to change it if you think otherwise.

Copy link
Contributor

Choose a reason for hiding this comment

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

GitHub teams can be mentioned also as suggested by @Souvikns for better transparency and easy tagging.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for reminding me. Also added Slack channel 👍

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.

I added some minor changes, and it looks good to go.

WORKING_GROUPS.md Outdated Show resolved Hide resolved
Co-authored-by: Fran Méndez <[email protected]>
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.

👌🏿

@fmvilas
Copy link
Member Author

fmvilas commented Feb 27, 2024

Awesome! Thanks for the changes, @thulieblack 🙌

@thulieblack
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 0361b9e into asyncapi:master Feb 27, 2024
8 checks passed
@fmvilas fmvilas deleted the add-working-groups branch February 27, 2024 12:53
Copy link
Member

@derberg derberg left a comment

Choose a reason for hiding this comment

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

I'm kinda missing in description information about meetings.

How do we see this happens? how we assure transparency? will we use current meetings framework?

working_groups:
# - name: Required. The name of the working group.
# description: Required. Describe what this working group is about.
# chairperson: Required. GitHub handle of the chairperson. Example: @Barbanio.
Copy link
Member

Choose a reason for hiding this comment

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

what if there are 2 or more chairpersons? better to have it as list imho, like in case of members

Copy link
Member

Choose a reason for hiding this comment

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

@fmvilas fyi, yeah, it was merged before I had a chance to look into it

Copy link
Member Author

Choose a reason for hiding this comment

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

Apologies. Just realized about your comments right now. Will open a separate PR.

# members: # Required to have at least 2 members who are not the same as the chairperson.
# - @fmvilas # GitHub handle of the member.
# slack_channel: # Required. The name of the Slack channel in the AsyncAPI workspace. Example: community_growth_wg
# okrs_url: https://example.com/xyz # Required. Link to a GitHub project, issue, or any other tool where the Working Group explains their objectives.
Copy link
Member

Choose a reason for hiding this comment

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

there's no single sentence about ORKs in WORKING_GROUPS.md - so the name of property can be pretty confusing. And why required? I could also add ORKs as a bullet list here in YAML to get it rendered in website. Maybe just call it goals and define it as a field where you can write goals or provide link?

and I immediately ask what is the difference with roadmap_url - why separate link?

Copy link
Member

Choose a reason for hiding this comment

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

@fmvilas fyi, yeah, it was merged before I had a chance to look into it

Copy link
Member Author

Choose a reason for hiding this comment

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

Apologies. Just realized about your comments right now. Will open a separate PR.

Copy link
Member Author

@fmvilas fmvilas Mar 30, 2024

Choose a reason for hiding this comment

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

Just created a PR: #1138. Regarding differences between objectives and roadmap, have a look at DX working group details. Long story short, objectives are for setting direction (the why), roadmap is exactly the issues/tasks (the what and how) you're going to be addressing in the foreseeable future (in their case 6 weeks).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants