Skip to content

[12.x] Keep API docs on website branch v12 - #9005

Merged
fhammerschmidt merged 1 commit into
maintenance/12.xfrom
fix/v12-api-docs-branch
Oct 24, 2026
Merged

fhammerschmidt merged 1 commit into
maintenance/12.xfrom
fix/v12-api-docs-branch

Conversation

@fhammerschmidt

Copy link
Copy Markdown
Member

The v12 maintenance workflow follows the website's default branch for API-docs checks and publication. Pin the checkout to v12 and push explicitly to HEAD:v12, so v12 releases keep updating v12 documentation after the website default moves to v13.

Follow-up to #8945; related to #8753.

Validation:

  • opam exec --switch=5.3.0 -- make passed.
  • Parsed the workflow YAML and executed its publication step against a temporary Git repository with default branch v13 and a detached v12 checkout; only v12 received the snapshot.
  • Validated the changelog fragment and rendered the commit record.

The maintenance workflow follows the website default branch. Pin its checkout and push to v12 so future website promotion keeps v12 maintenance snapshots on their own documentation branch.

Follow up on #8945 and #8753.

Signed-off-by: Florian Hammerschmidt <florianh89@gmail.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-23T20:38:31.373686Z
We couldn’t complete this review. Comment @codex review to try again.
4e905f6 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@fhammerschmidt
fhammerschmidt requested a review from cknitt October 23, 2026 20:38
@pkg-pr-new

pkg-pr-new Bot commented Oct 23, 2026

Copy link
Copy Markdown

Open in StackBlitz

rescript

npm i https://pkg.pr.new/rescript@9005

@rescript/darwin-arm64

npm i https://pkg.pr.new/@rescript/darwin-arm64@9005

@rescript/darwin-x64

npm i https://pkg.pr.new/@rescript/darwin-x64@9005

@rescript/linux-arm64

npm i https://pkg.pr.new/@rescript/linux-arm64@9005

@rescript/linux-x64

npm i https://pkg.pr.new/@rescript/linux-x64@9005

@rescript/runtime

npm i https://pkg.pr.new/@rescript/runtime@9005

@rescript/win32-x64

npm i https://pkg.pr.new/@rescript/win32-x64@9005

commit: 4e905f6

@fhammerschmidt
fhammerschmidt merged commit 85120d4 into maintenance/12.x Oct 24, 2026
27 checks passed
@fhammerschmidt
fhammerschmidt deleted the fix/v12-api-docs-branch branch October 24, 2026 07: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