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
8 changes: 8 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,14 @@ repos:
- id: check-yaml
# catch git merge/rebase problems
- id: check-merge-conflict
- repo: https://github.com/python-jsonschema/check-jsonschema
rev: 0.28.5
hooks:
- id: check-github-workflows
# validate sub-teams schema
- id: check-jsonschema
files: ^teams/.*/.*\.ya?ml$
args: [--schemafile, teams/sub-teams-schema.json]
Comment on lines +28 to +30
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

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.8.0
hooks:
Expand Down
27 changes: 27 additions & 0 deletions teams/dynamic/coc-committee.yml
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

Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name:
description: The Code of Conduct Committee
charter: dynamic
requirements:
scopes:
codeowners:
other:
- https://github.com/conda-conduct
- https://matrix.to/#/!jLRCFFbpabFtdYvfeO:matrix.org # private
links:
- https://github.com/conda/governance/issues/46
- https://github.com/conda/governance/pull/49
- https://github.com/conda/governance/pull/121
- https://github.com/conda/governance/pull/158
members:
beeankha: https://github.com/conda/governance/pull/49
csoja: https://github.com/conda/governance/issues/46
dashagurova: https://github.com/conda/governance/pull/158
ericdill: https://github.com/conda/governance/issues/46
ForgottenProgramme: https://github.com/conda/governance/issues/46
jaimergp: https://github.com/conda/governance/pull/121
kathatherine: https://github.com/conda/governance/pull/49
kenodegard: https://github.com/conda/governance/issues/46
emeritus:
ocefpaf: https://github.com/conda/governance/pull/158
tnabtaf: https://github.com/conda/governance/pull/121
22 changes: 22 additions & 0 deletions teams/dynamic/communications.yml
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

Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: communications
description:
charter: dynamic
requirements:
scopes:
codeowners:
- conda/communications
other:
- https://conda.discourse.group/
- https://www.linkedin.com/company/condacommunity
- https://fosstodon.org/@conda
- https://matrix.to/#/#conda-communications:matrix.org # public
- https://x.com/condaproject
- https://bsky.app/profile/condaproject.bsky.social
links:
- https://github.com/conda/governance/issues/53
members:
jaimergp: https://github.com/conda/governance/issues/81
jezdez: https://github.com/conda/governance/issues/81
emeritus:
tnabtaf: https://github.com/conda/governance/issues/81
36 changes: 36 additions & 0 deletions teams/project/builds-tools.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: builds-tools
description: Team to work on conda-build and other build tools in the conda ecosystem
charter: project
requirements:
scopes:
codeowners:
- conda-incubator/conda-recipe-manager
- conda/conda-build
other:
- https://matrix.to/#/#conda-build-tools:matrix.org # public
links:
- https://github.com/conda/conda-build/issues/4698
- https://github.com/conda/governance/issues/128
members:
beeankha: https://github.com/conda/conda-build/issues/4698
beckermr:
chenghlee: https://github.com/conda/conda-build/issues/4698
dbast: https://github.com/conda/conda-build/issues/4698
dholth: https://github.com/conda/conda-build/issues/4698
isuruf: https://github.com/conda/conda-build/issues/4698
jaimergp: https://github.com/conda/conda-build/issues/4698
jakirkham: https://github.com/conda/conda-build/issues/4698
JeanChristopheMorinPerso: https://github.com/conda/conda-build/issues/4698
jezdez: https://github.com/conda/conda-build/issues/4698
kenodegard: https://github.com/conda/conda-build/issues/4698
marcelotrevisani: https://github.com/conda/conda-build/issues/4698
mbargull: https://github.com/conda/conda-build/issues/4698
msarahan: https://github.com/conda/conda-build/issues/4698
ryanskeith: https://github.com/conda/conda-build/issues/4698
travishathaway: https://github.com/conda/conda-build/issues/4698
wolfv: https://github.com/conda/conda-build/issues/4698
schuylermartin45: https://github.com/conda/governance/issues/128
emeritus:
Callek:
danpetry: https://anaconda.slack.com/archives/C05B9MPGR4Y/p1713452484809289
30 changes: 30 additions & 0 deletions teams/project/constructor.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: constructor
description: The team maintaining constructor and related projects
charter: project
requirements:
scopes:
codeowners:
- conda/conda-pack
- conda/constructor
- conda/menuinst
- conda/conda-standalone
- conda/nsis-mirror
other:
links:
- https://github.com/conda/governance/issues/53
members:
bkreider: https://github.com/conda/governance/issues/53
chenghlee: https://github.com/conda/governance/issues/53
goanpeca: https://github.com/conda/governance/issues/53
hmaarrfk: https://github.com/conda/governance/issues/53 # missing from team?
hoechenberger: https://github.com/conda/governance/issues/53
isuruf: https://github.com/conda/governance/issues/53
jaimergp: https://github.com/conda/governance/issues/53
jezdez: https://github.com/conda/governance/issues/53
larsoner: https://github.com/conda/governance/issues/53
marcoesters: # missing record of decision?
mcg1969: https://github.com/conda/governance/issues/53 # missing from team?
pseudoyim: https://github.com/conda/governance/issues/53
travishathaway: # missing record of decision?
emeritus:
22 changes: 22 additions & 0 deletions teams/project/docs.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: docs
description: The team working on all-things docs
charter: project
requirements:
scopes:
codeowners:
- conda/conda
- conda/conda-build
- conda/conda-docs
- conda/constructor
- conda/infrastructure
- conda/menuinst
other:
links:
- https://github.com/conda/conda-docs/issues/780
members:
kathatherine: https://github.com/conda/conda-docs/issues/780
travishathaway: https://github.com/conda/conda-docs/issues/780
emeritus:
jezdez:
tnabtaf:
17 changes: 17 additions & 0 deletions teams/static/cla-reviewers.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: cla-reviewers
description: The team reviewing incoming CLA documents
charter: static
requirements: All team members must only consists of steering council members for compliance reasons.
scopes:
codeowners:
- conda/cla
other:
links:
- https://github.com/conda/governance/issues/136
members:
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
emeritus:
23 changes: 23 additions & 0 deletions teams/static/infrastructure.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# yaml-language-server: $schema=../sub-teams-schema.json
name: infrastructure
description: The team working on conda community infrastructure
charter: static
requirements: Any steering council member may join without approval
scopes:
codeowners:
- conda/.github
- conda/actions
- conda/infrastructure
- conda/issue-tracker
other:
- https://github.com/conda-bot
- https://matrix.to/#/!tpEQSUGYWHNdgpDFNc:matrix.org # private
links:
- https://github.com/conda/governance/issues/84
members:
beckermr: https://github.com/conda/governance/issues/84
chenghlee: # missing record of decision?
dbast: https://github.com/conda/governance/issues/84
jezdez: https://github.com/conda/governance/issues/84
kenodegard: https://github.com/conda/governance/issues/84
emeritus:
81 changes: 81 additions & 0 deletions teams/sub-teams-schema.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
{
"$schema": "https://json-schema.org/draft/2020-12/schema",
"title": "Conda Sub-Teams",
"type": "object",
"properties": {
"name": {
"type": ["string", "null"],
"description": "The team name in GitHub"
},
"description": {
"type": ["string", "null"],
"description": "The team description in GitHub"
},
"charter": {
"type": "string",
"description": "The team charter type, see [dynamic](https://github.com/conda/governance#dynamic-charter), [static](https://github.com/conda/governance#static-charter), and [project](https://github.com/conda/governance#project-teams-details) requirements",
"enum": ["dynamic", "static", "project"]
},
"requirements": {
"type": ["string", "null"],
"description": "Special requirements for team membership"
},
"scopes": {
"type": ["object", "null"],
"properties": {
"codeowners": {
"type": ["array", "null"],
"items": {
"type": "string"
},
"description": "The GitHub projects this team owns"
},
"other": {
"type": ["array", "null"],
"items": {
"type": "string",
"format": "uri"
},
"description": "Other responsibilities of this team"
}
},
"required": ["codeowners", "other"],
"additionalProperties": false,
"description": "Team responsibilities"
},
"links": {
"type": "array",
"items": {
"type": "string",
"format": "uri"
},
"description": "Important links, e.g. the issue/PR proposing the team creation"
},
"members": {
"type": "object",
"patternProperties": {
".*": {
"type": "string",
"format": "uri"
}
},
"minProperties": 1,
"additionalProperties": false,
"description": "Maps username to a record of the decision adding them to the team"
},
"emeritus": {
"type": ["object", "null"],
"patternProperties": {
".*": {
"type": "string",
"format": "uri"
}
},
"minProperties": 1,
"additionalProperties": false,
"description": "Maps username to a record of the decision removing them from the team"
}
},
"required": ["name", "description", "charter", "requirements", "scopes", "links", "members", "emeritus"],
"additionalProperties": false
}