feat: add flag to allow cross-organization teams during org transitions #228
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jasonwbarnett
changed the title
jasonwbarnett
force-pushed
the
jwb/allow-cross-repo
branch
3 times, most recently
from
c41007a to
f0d5edf
Compare
This allows teams from different organizations to be valid owners, which is useful during repository transitions between organizations (e.g., from altana-poc to altana-tech). When OWNER_CHECKER_ALLOW_CROSS_ORG_TEAMS is set to true, the validator will: - Skip the check that ensures teams belong to the repository's organization - Validate that teams actually exist in their respective organizations - Skip permission checks for cross-org teams (can't verify permissions across orgs) This allows CODEOWNERS files to reference teams in the target organization while the repository is temporarily in a different organization for testing. Teams are still validated for existence, ensuring typos are caught. Updates: - Add AllowCrossOrgTeams field to ValidOwnerConfig - Refactor team validation to fetch teams from the appropriate org - Skip permission checks for cross-org teams - Add documentation for the new environment variable - Update GitHub Action definition with new input parameter - Add unit tests for the new configuration flag 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
jasonwbarnett
force-pushed
the
jwb/allow-cross-repo
branch
from
f0d5edf to
db9bc0f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Changes proposed in this pull request:
OWNER_CHECKER_ALLOW_CROSS_ORG_TEAMSenvironment variable to allow teams from different organizations to be valid CODEOWNERSold-org/repotonew-org/repo)falseto maintain backward compatibilityRelated issue(s)
N/A - Feature request to support organization transitions where teams are defined in the target organization but the repository temporarily resides in a different organization for testing purposes.