Skip to content

chore: add ownership-guard action#6

Merged
flaticols merged 1 commit into
mainfrom
test/ownership-guard
Apr 25, 2026
Merged

chore: add ownership-guard action#6
flaticols merged 1 commit into
mainfrom
test/ownership-guard

Conversation

@flaticols
Copy link
Copy Markdown
Owner

Adds flaticols/ownership-guard@v2.0.0 to enforce cross-team ownership awareness on PRs.

What this does

  • Adds .ownership file defining the core team with ownership over internal/** and cmd/**
  • Adds .github/workflows/ownership.yml that runs on every PR

Testing

This draft PR itself is the test — the ownership workflow should fire and post a warning comment if the PR author is not in the core team.

.ownership syntax

team: core
@flaticols/core

=ownership:
internal/**
cmd/**

@flaticols flaticols force-pushed the test/ownership-guard branch 3 times, most recently from dd6a4dc to 00df1cd Compare April 25, 2026 19:01
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

Important

internal is part of the analysis engine owned by core. @flaticols, please coordinate with @flaticols/core — changes here affect the core refactoring logic.

@flaticols flaticols force-pushed the test/ownership-guard branch 2 times, most recently from 8d18a47 to 965e006 Compare April 25, 2026 19:08
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

Note

internal/tui is owned by the tui team. @flaticols, loop in @flaticols/core — UI changes should be reviewed for keyboard, rendering, and accessibility consistency.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

Warning

nix is owned by infra. @flaticols, please align with @flaticols/core before merging — nix changes affect the binary packaging and release pipeline.

@flaticols flaticols force-pushed the test/ownership-guard branch 3 times, most recently from 4fa71db to 12fd0e1 Compare April 25, 2026 19:20
@flaticols flaticols force-pushed the test/ownership-guard branch from 12fd0e1 to df0d7d0 Compare April 25, 2026 19:25
@flaticols flaticols marked this pull request as ready for review April 25, 2026 19:26
@flaticols flaticols merged commit 23425fa into main Apr 25, 2026
1 of 3 checks passed
@flaticols flaticols deleted the test/ownership-guard branch April 25, 2026 19:31
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