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

Documenting sub-teams as YAML files (and verified using a JSON Schema) #147

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

kenodegard
Copy link
Contributor

@kenodegard kenodegard commented Jun 13, 2024

Description

We now have a number of sub-teams and it's getting a little hard to keep track of everything. This is an attempt to create a more formalized way of documenting what the teams look like and what their responsibilities are.

Closes #86

@kenodegard kenodegard requested a review from a team as a code owner June 13, 2024 08:27
Copy link
Member

@jezdez jezdez left a comment

Choose a reason for hiding this comment

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

I haven't so far checked all the members based on existing teams FWIW, but just wanted to be clear that not all the teams listed so far are actual subteams per governance policy, but some are project teams, which has a difference in that subteams act on behalf of a static charter, while project teams are dedicated to specific projects (or groups of related projects)

teams/dynamic/communications.yml Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

communications doesn't have a dynamic charter, but has a static charter as shown in #81

Copy link
Contributor Author

@kenodegard kenodegard Jun 13, 2024

Choose a reason for hiding this comment

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

It's listed as dynamic here: #81 (comment)

Screenshot 2024-06-14 at 09 24 44

Copy link
Member

Choose a reason for hiding this comment

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

Also not a dynamic charter, see the founding issue

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's listed as dynamic here: #46 (comment)

Screenshot 2024-06-14 at 09 24 01

teams/dynamic/constructor.yml Outdated Show resolved Hide resolved
teams/dynamic/constructor.yml Outdated Show resolved Hide resolved
chenghlee: https://github.com/conda/governance/issues/136
jaimergp: https://github.com/conda/governance/issues/136
jezdez: https://github.com/conda/governance/issues/136
travishathaway: https://github.com/conda/governance/issues/136
Copy link
Member

Choose a reason for hiding this comment

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

Travis participating is a problem for the requirements

teams/static/infrastructure.yml Outdated Show resolved Hide resolved
codeowners: null
other:
GitHub Account: https://github.com/conda-conduct
Matrix (private): https://matrix.to/#/!jLRCFFbpabFtdYvfeO:matrix.org
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hm does this link also act as an invitation to the private channel?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, that appears to be the case, though it's tough to tell 100% since I'm already part of that channel. I opened this link in an incognito Chrome tab and it allowed me to open the channel in Element (there were also other app options offered) but as I said I'm already a member so I'm not sure if there would be some sort of block from me joining the channel otherwise.

Copy link
Member

@beeankha beeankha Jun 17, 2024

Choose a reason for hiding this comment

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

Attached are screenshots to display what happens when attempting to access a private channel URL/invite link that I am not a member of:

Screenshot 2024-06-17 at 10 27 57 AM

element

Screenshot 2024-06-17 at 10 18 02 AM

Looks like there is a guardrail in place that prevents folks from joining private channels that they weren't invited to.

wolfv: https://github.com/conda/conda-build/issues/4698
schuylermartin45: https://github.com/conda/governance/issues/128
emeritus:
Callek: null
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Callek is no longer in the GH team but I haven't found a record of when he left the team

Copy link

@Callek Callek Jun 17, 2024

Choose a reason for hiding this comment

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

@kenodegard
I left the team org Monday May 6, 2024. (Jannis was cleaning up team memberships, and I didn't need it)
image

Copy link

Choose a reason for hiding this comment

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

I'm also ok if you don't list me as Emeritus, I never considered myself active in the "team" and my participation in our meetings, was mainly just to hear about problems and upcoming changes.

Comment on lines 21 to 22
jezdez: null
tnabtaf: null
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Did not find a record for either @jezdez or tnabtaf leaving this team

Comment on lines +18 to +20
- id: check-jsonschema
files: ^teams/.*/.*\.ya?ml$
args: [--schemafile, teams/sub-teams-schema.json]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Verifies whether the sub-team YAMLs are valid!

I recommend enabling pre-commit CI

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Xref #156

teams/dynamic/coc-committee.yml Outdated Show resolved Hide resolved
teams/dynamic/coc-committee.yml Outdated Show resolved Hide resolved
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.

Record-keeping for subteams
4 participants