Skip to content

Conversation

@bryangingechen
Copy link
Contributor

@bryangingechen bryangingechen commented Nov 10, 2025

cf. https://github.com/leanprover-community/mathlib4/actions/runs/19220373453/job/54937283962

Follow-up to #31422


Do we also need a CI step to catch this sort of thing in the future?

@github-actions
Copy link

PR summary d6b7d98695

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).

@adomani
Copy link
Collaborator

adomani commented Nov 10, 2025

Thanks! 🐙

bors merge

@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Nov 10, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title fix: broken lakefile.lean [Merged by Bors] - fix: broken lakefile.lean Nov 10, 2025
@mathlib-bors mathlib-bors bot closed this Nov 10, 2025
@mathlib-bors mathlib-bors bot deleted the bryangingechen-patch-2 branch November 10, 2025 09:23
mathlib-bors bot pushed a commit that referenced this pull request Nov 20, 2025
This adds a little bit more monitoring; would have caught the failures fixed by #31457 a little earlier.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants