Skip to content

docs: add pre-commit usage with ADO Artifact feeds#39

Open
temporaer wants to merge 1 commit into
mainfrom
docs/pre-commit-usage
Open

docs: add pre-commit usage with ADO Artifact feeds#39
temporaer wants to merge 1 commit into
mainfrom
docs/pre-commit-usage

Conversation

@temporaer

@temporaer temporaer commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Adds a Usage with pre-commit section to the README explaining how to route pre-commit's Python package installations through an ADO Artifact feed for supply-chain compliance.

Approach

Hook repositories are still cloned from GitHub via git (pinned by commit SHA) — only the pip install step is routed through the feed via keyring authentication.

What's documented

  • Prerequisites: keyring tool installation + pip configuration (pip.conf or env vars)
  • Hook configuration: standard repo: entries with commit SHA pinning (not repo: local)
  • Limitations: only language: python hooks route through the feed; git clones still go directly to GitHub
  • Cache tip: pre-commit clean when credentials expire

No code changes

The keyring backend already handles pip authentication. Pre-commit calls pip in a subprocess, and pip respects PIP_INDEX_URL + PIP_KEYRING_PROVIDER=subprocess.

Copilot AI review requested due to automatic review settings June 16, 2026 18:21

This comment was marked as outdated.

@temporaer temporaer force-pushed the docs/pre-commit-usage branch 2 times, most recently from 9a93339 to 0b381b0 Compare June 16, 2026 18:57
Explain how to install private pre-commit hooks (Python packages) hosted
on Azure DevOps Artifact feeds using the repo:local pattern with
additional_dependencies and pip keyring authentication.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@temporaer temporaer force-pushed the docs/pre-commit-usage branch from 0b381b0 to 5b4b019 Compare June 16, 2026 19:01
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.

4 participants