chore(deps): bump huggingface/doc-builder/.github/workflows/build_pr_documentation.yml from 7ccf6c02dc2e3aea1b07f6668f783b20c3bf407c to cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3 - #1121
Conversation
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Alignment Review Report
Dependabot bump of the SHA-pinned, HF-owned huggingface/doc-builder reusable workflow in build_pr_documentation.yml: 7ccf6c02 → 9978a413 (single uses: line).
Automated Checks
- Lint: PASS (N/A for scope) — 0 Python files changed; the CI lint job only covers
src/+tests/(neither touched) and TOML/YAML aren't linted. (lint.sherrors only becauseuvisn't installed on the review VM — pre-existing, unrelated to this PR.) - Debug code: CLEAN — all
check-debug.shhits are pre-existingsrc/openenv/core/mcp_client.pyprints +cli/serve/__main__/test_local_docker_providerTODOs; none are in this diff. - YAML: valid (
yaml.safe_loadOK).
Supply-chain / caller-contract verification
- New SHA legitimate & current:
7ccf6c02...9978a413is a clean fast-forward (ahead 15 / behind 0);9978a413...main==identical, so the new pin is doc-builder's currentmainHEAD and the trailing# maincomment stays accurate. New tip = doc-builder #826 "Comment out schedule for search engine population". - Reusable-workflow interface changed but backward-compatible: unlike most prior bumps,
build_pr_documentation.ymlis in the compare's changed files. Its only delta is doc-builder #808 — an optionalpython_versioninput (type: string, norequired:, nodefault:) plus auv venv …line that is byte-identical when the input is unset. Theworkflow_callrequired inputs at9978a413remain exactlycommit_sha/pr_number/package. Our caller supplies all three (+version_tag_suffix: "") and does not passpython_version, so it keeps the runner's system Python — no new required input, nothing breaks.build_prhas nosecrets:block and our caller passes none. - Artifact contract intact: the patch doesn't touch the upload step, so the
doc-build-artifacthandoff toupload_pr_documentation.ymlis unchanged. - Security: still a full 40-char SHA pin; no hardcoded credentials.
- Mergeability:
git merge-tree origin/main <head>is clean (origin/main tipced9d73c== PR base == merge-base → clean 1-commit fast-forward); no stale-branch conflict.
Open RFCs Context
rfcs/ = 000/001/002/003/005/008 (In Review), 004-rubrics (no Status header), 010 (Draft). None govern CI / GitHub Actions / dependency management / the docs-build pipeline → no RFC conflicts.
Tier 1: Fixes Required
None.
Tier 2: Alignment Discussion
Principle Conflicts — None identified. A CI workflow SHA bump touches no runtime API / reward / client-server / MCP-WS / credential surface; full-SHA pinning + dependabot freshness aligns with the reproducibility/security principle.
RFC Conflicts — None identified.
Non-blocking note (informational)
- Sibling pin skew (expected): the sibling doc-builder pins are still at
7ccf6c02—build_documentation.yml:12(build_main) andupload_pr_documentation.yml:11. Dependabot tracks each reusable-workflow path as a separate dependency, so temporary skew is expected and reconverges as the sibling PRs (or acodex/dependabot-core-*aggregate) land. cc @burtenshaw (owner of.github/). Not a blocker.
Summary
- 0 mechanical issues to fix
- 0 alignment points for human review
- 0 RFC conflicts
Clean, low-risk first-party bump. LGTM.
Sent by Cursor Automation: Pre-review
| build: | ||
| if: github.event.pull_request.draft == false | ||
| uses: huggingface/doc-builder/.github/workflows/build_pr_documentation.yml@7ccf6c02dc2e3aea1b07f6668f783b20c3bf407c # main | ||
| uses: huggingface/doc-builder/.github/workflows/build_pr_documentation.yml@9978a4134f9e245a927f8d85f73c026609b38b47 # main |
There was a problem hiding this comment.
Verified this bump. 7ccf6c02…9978a413 is a clean fast-forward (ahead 15 / behind 0) and 9978a413 == doc-builder main HEAD, so the # main comment stays accurate.
The reusable workflow's only interface change across the delta is doc-builder #808's optional python_version input (no required:, no default:; the uv venv line is byte-identical when it's unset). Required workflow_call inputs remain commit_sha/pr_number/package — all supplied by this caller — so the contract is unbroken and the runner's system Python is retained. Full 40-char SHA pin preserved.
873ef4a to
69376d3
Compare
…documentation.yml Bumps [huggingface/doc-builder/.github/workflows/build_pr_documentation.yml](https://github.com/huggingface/doc-builder) from 7ccf6c02dc2e3aea1b07f6668f783b20c3bf407c to cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3. - [Release notes](https://github.com/huggingface/doc-builder/releases) - [Changelog](https://github.com/huggingface/doc-builder/blob/main/RELEASE.md) - [Commits](huggingface/doc-builder@7ccf6c0...cf20b09) --- updated-dependencies: - dependency-name: huggingface/doc-builder/.github/workflows/build_pr_documentation.yml dependency-version: 9978a4134f9e245a927f8d85f73c026609b38b47 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
69376d3 to
d661e9e
Compare
|
Looks like huggingface/doc-builder/.github/workflows/build_pr_documentation.yml is up-to-date now, so this is no longer needed. |


Bumps huggingface/doc-builder/.github/workflows/build_pr_documentation.yml from 7ccf6c02dc2e3aea1b07f6668f783b20c3bf407c to cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3.
Changelog
Sourced from huggingface/doc-builder/.github/workflows/build_pr_documentation.yml's changelog.
Commits
cf20b09Revert "Comment out schedule for search engine population (#826)" (#827)9978a41Comment out schedule for search engine population (#826)c2d27f6Fix vectorless Meilisearch document payload (#825)953aa44Add vectorless full-text docs ingestion (#824)1b16dacRemove setup.py in favor of pyproject.toml (#816)bcd143eCheck anchors in links (#820)68667a5fix(kit): accept a lowercase region in language codes (pt-br) (#823)0ab9ea0Ship a pre-commit hook for doc-builder style (#818)4a56e30Escape HTML and strip ANSI codes in notebook-to-mdx text outputs (#819)da05e70delete-old-prs: use a trusted publisher instead of the HF_ACCESS_TOKEN secret...Note
Low Risk
Single CI workflow pin change with no application or runtime code touched; risk is limited to PR doc preview/build behavior if the upstream workflow regresses.
Overview
Updates the PR documentation GitHub Actions workflow to call a newer pinned revision of Hugging Face’s reusable
build_pr_documentation.yml(7ccf6c0→cf20b09). Inputs (commit_sha,pr_number,package: openenv, etc.) are unchanged; only the upstream workflow implementation moves forward (e.g. doc-builder fixes around link anchors, notebook MDX output, and related tooling from that commit range).Reviewed by Cursor Bugbot for commit d661e9e. Bugbot is set up for automated code reviews on this repo. Configure here.