Skip to content

Give the reference adapters a code owner - #513

Merged
rocklambros merged 1 commit into
integrationfrom
chore/codeowners-adapters
Nov 5, 2026
Merged

rocklambros merged 1 commit into
integrationfrom
chore/codeowners-adapters

Conversation

@rocklambros

Copy link
Copy Markdown
Collaborator

What changed

This adds an /adapters/ entry to CODEOWNERS that names Bar Kaduri as the owner of the reference adapters, which PR #22 merged into integration without an entry of their own. Ariel Fogel and Rock Lambros are listed with her, because GitHub does not let an author approve her own pull request, and with Bar as the only owner every change she made to the adapters would need an admin bypass.

Which issue does this implement

Refs #22

Base branch

  • integration, because this touches the specification, schemas, a reference
    implementation, an adapter, tests, or CI

Type of change

  • Governance (licensing, security policy, contributor docs)

I tested this

  • uv run pytest tests/test_roadmap_model_roster.py tests/test_governance_files.py tests/test_dependabot_config.py passes (34 passed)

Checklist

  • Commits are signed off with git commit -s (required by the DCO)
  • No secrets, tokens, or internal URLs in the diff

PR #22 put the adapters under adapters/ with no CODEOWNERS entry, so
they fell to the repository default. Bar Kaduri owns them. Ariel Fogel
and Rock Lambros are listed with her because an author cannot approve
her own pull request.

Signed-off-by: rocklambros <rock@rockcyber.com>
@github-project-automation github-project-automation Bot moved this to Needs triage in ACS Project Tracker Nov 5, 2026
@rocklambros
rocklambros merged commit 41da52d into integration Nov 5, 2026
12 checks passed
@rocklambros
rocklambros deleted the chore/codeowners-adapters branch November 5, 2026 21:31
@github-project-automation github-project-automation Bot moved this from Needs triage to Done in ACS Project Tracker Nov 5, 2026
@github-actions github-actions Bot mentioned this pull request Nov 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant