Skip to content

[Merged by Bors] - chore: support revised nightly toolchains (nightly-YYYY-MM-DD-revK)#35220

Closed
kim-em wants to merge 1 commit intomasterfrom
kim/nightly-rev-support
Closed

[Merged by Bors] - chore: support revised nightly toolchains (nightly-YYYY-MM-DD-revK)#35220
kim-em wants to merge 1 commit intomasterfrom
kim/nightly-rev-support

Conversation

@kim-em
Copy link
Contributor

@kim-em kim-em commented Feb 13, 2026

This PR updates nightly workflow scripts to handle revised nightly toolchains (nightly-YYYY-MM-DD-revK). When multiple nightlies are released for the same date, the revision suffix ensures correct ordering and matching in tag grep patterns, version extraction regexes, and deduplication checks.

Changes:

  • nightly_bump_and_merge.yml: update tag grep pattern and use sort -rV for correct version ordering
  • nightly_detect_failure.yml: update JS/Python regexes and warning message to handle -revK suffix

Companion to leanprover/lean4#12461

🤖 Prepared with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

PR summary c72b0ec472

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@github-actions github-actions bot added the CI Modifies the continuous integration setup or other automation label Feb 13, 2026
github-merge-queue bot pushed a commit to leanprover/lean4 that referenced this pull request Feb 13, 2026
…12461)

This PR adds support for manually re-releasing nightlies when a build
issue or critical fix requires it. When a `workflow_dispatch` triggers
the nightly release job and a `nightly-YYYY-MM-DD` tag already exists,
the CI now creates `nightly-YYYY-MM-DD-rev1` (then `-rev2`, etc.)
instead of silently skipping.

### Lake `ToolchainVer`

- Extend `ToolchainVer.nightly` with an optional `rev : Option Nat`
field
- Parse `-revK` suffixes from nightly tags in `ofString`
- Ordering: `nightly-YYYY-MM-DD` < `nightly-YYYY-MM-DD-rev1` < `-rev2` <
`nightly-YYYY-MM-DD+1`
- Round-trip: `toString (ofString s) == s` for both variants

### CI workflow

- "Set Nightly" step probes existing tags on `workflow_dispatch` to find
next available `-revK`
- Scheduled nightlies retain existing behavior (skip if commit already
tagged)
- Changelog grep updated from `nightly-[-0-9]*` to `nightly-[^ ,)]*` to
match `-revK` suffixes

### `lean-bisect`

- Updated `NIGHTLY_PATTERN` regex, sort key, error messages, and help
text

### Companion PRs

- leanprover-community/mathlib4#35220: update
`nightly_bump_and_merge.yml` tag grep and `nightly_detect_failure.yml`
warning message
-
leanprover-community/leanprover-community.github.io#787:
update `tags_and_branches.md` documentation

🤖 Prepared with Claude Code

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@kim-em
Copy link
Contributor Author

kim-em commented Feb 13, 2026

I'm merging this one myself, see #mathlib reviewers > requests for speedy review/delegation @ 💬

bors merge

@mathlib-triage mathlib-triage bot added the ready-to-merge This PR has been sent to bors. label Feb 13, 2026
mathlib-bors bot pushed a commit that referenced this pull request Feb 13, 2026
…35220)

This PR updates nightly workflow scripts to handle revised nightly toolchains (`nightly-YYYY-MM-DD-revK`). When multiple nightlies are released for the same date, the revision suffix ensures correct ordering and matching in tag grep patterns, version extraction regexes, and deduplication checks.

Changes:
- `nightly_bump_and_merge.yml`: update tag grep pattern and use `sort -rV` for correct version ordering
- `nightly_detect_failure.yml`: update JS/Python regexes and warning message to handle `-revK` suffix

Companion to leanprover/lean4#12461

🤖 Prepared with Claude Code
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Feb 13, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: support revised nightly toolchains (nightly-YYYY-MM-DD-revK) [Merged by Bors] - chore: support revised nightly toolchains (nightly-YYYY-MM-DD-revK) Feb 13, 2026
@mathlib-bors mathlib-bors bot closed this Feb 13, 2026
@mathlib-bors mathlib-bors bot deleted the kim/nightly-rev-support branch February 13, 2026 02:24
kim-em added a commit to kim-em/mathlib4 that referenced this pull request Feb 14, 2026
…eanprover-community#35220)

This PR updates nightly workflow scripts to handle revised nightly toolchains (`nightly-YYYY-MM-DD-revK`). When multiple nightlies are released for the same date, the revision suffix ensures correct ordering and matching in tag grep patterns, version extraction regexes, and deduplication checks.

Changes:
- `nightly_bump_and_merge.yml`: update tag grep pattern and use `sort -rV` for correct version ordering
- `nightly_detect_failure.yml`: update JS/Python regexes and warning message to handle `-revK` suffix

Companion to leanprover/lean4#12461

🤖 Prepared with Claude Code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Modifies the continuous integration setup or other automation ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant