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

Add CODEOWNERS team for oqs-demos #140

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

SWilson4
Copy link
Member

See open-quantum-safe/oqs-demos#350 and open-quantum-safe/oqs-demos#351. Leaving as draft pending feedback on those issues/PRs.

Unconditionally, changes to config.yaml must

  • be approved by 2 members of the OQS TSC
  • not violate permissions documented in GOVERNANCE.md files for sub projects where such files exist

The following goals apply to changes to the file config.yaml with exceptions possible, as long as the rationale for the exception is documented by comments in the file:

  • all sub projects should be treated identically wrt roles & responsibilities as per the detailed list below
  • teams/team designations are to be used wherever possible; using personal GH handles should only be used in team definitions
  • Admin changes to the file must be documented by comments as to the rationale of the change

All the following conditions hold for permissions set in config.yaml:

  • sub project maintainers have admin rights on the sub projects
  • OQS and sub project release managers have maintainer rights on the sub projects but can themselves set/reset branch protection rules limiting write access to sensitive branches
  • sub project committers have write rights on all branches of the sub projects but can request branch protection rules limiting this
  • sub project contributors (incl. code owners) have write rights on all branches except main on those sub projects
  • OQS and sub project triage actors have triage rights on all branches of the sub projects
  • OQS maintainers and LF admins have admin rights on the organization (e.g., org-wide secret management) as well as maintenance rights on the team configurations

Copy link

clowarden bot commented Feb 12, 2025

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team oqs-demos-codeowners has been added
    • Maintainers
      • baentsch
    • Members
      • ajbozarth
      • bhess
      • pi-314159

🔸 Please review the changes detected as they will be applied immediately once this PR is merged 🔸

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.

1 participant