Skip to content

Allow forked repos to be checked out for mapping#2243

Open
KartikP wants to merge 1 commit intomasterfrom
kp/forked-layer-mapping
Open

Allow forked repos to be checked out for mapping#2243
KartikP wants to merge 1 commit intomasterfrom
kp/forked-layer-mapping

Conversation

@KartikP
Copy link
Collaborator

@KartikP KartikP commented Jan 12, 2026

Uses pull_request_target for both vision branches and forked vision branches while avoiding running untrusted code with secret access.

  1. ref: ${{ github.base_ref }} checks out master from brain-score/vision regardless of internal vs fork PR. Ensures dependencies come from trusted code.

  2. Existing Get Changed Files step continues to work for both. GitHub API knows about all PRs regardless of source.

  3. Existing GitHub action expressions continue to work for both.

@mike-ferguson mike-ferguson self-requested a review January 21, 2026 17:50
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.

2 participants