Skip to content

fix(repo-policy-sync): preserve Git auth for pre-commit hooks - #38

Merged
AlexanderLanin merged 1 commit into
eclipse-score:mainfrom
etas-contrib:bugfix/repo-policy-sync-pre-commit-auth
Sep 11, 2026
Merged

AlexanderLanin merged 1 commit into
eclipse-score:mainfrom
etas-contrib:bugfix/repo-policy-sync-pre-commit-auth

Conversation

@AlexanderLanin

Copy link
Copy Markdown
Member

Why

Apply mode runs repository-controlled pre-commit hooks with an isolated home directory and reduced environment. This currently hides the global Git URL rewrites installed by the apply workflow, so hooks that fetch private Bazel dependencies fail with Git authentication errors before the policy PR can be created.

What changed

  • copy only global url.*.insteadOf entries into a temporary Git configuration for pre-commit
  • continue excluding GH_TOKEN, GITHUB_TOKEN, user identity, aliases, credential helpers, and unrelated Git configuration
  • document the authentication hand-off and add regression coverage for authenticated nested Git fetches

@AlexanderLanin
AlexanderLanin merged commit 2f46bfa into eclipse-score:main Sep 11, 2026
3 checks passed
@AlexanderLanin
AlexanderLanin deleted the bugfix/repo-policy-sync-pre-commit-auth branch September 11, 2026 12:34
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