You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Full sync always reads configs from the default branch of the admin repo #1019
Scheduled syncs (CRON) and full syncs (full-sync.js) always read settings.yml, suborg and repo configs from the default branch of the admin repo. syncInstallation in index.js builds its context without a ref, so ConfigManager and Settings fall back to the default branch.
There is no way to point a full sync at a branch of the admin repo. That matters mostly for dry runs: with FULL_SYNC_NOP=true you would want to run the plan against a config branch before merging it, but the sync silently uses the default branch instead, so the plan tells you nothing about your proposed change. @seanuleh ran into exactly this in the #818 discussion ("this isn't even running against the PR ref... only against the default branch of ADMIN_REPO").
Webhook-triggered PR checks do this correctly (the check run passes pull_request.head.ref); it's only the scheduled/CLI path that can't.
Proposal: a CONFIG_REF environment variable that syncInstallation passes through as the ref, defaulting to today's behavior when unset.
Activity