Skip to content

feat: check that the mathlib options exist#25407

Closed
adomani wants to merge 4 commits intomasterfrom
adomani/check_options_exist
Closed

feat: check that the mathlib options exist#25407
adomani wants to merge 4 commits intomasterfrom
adomani/check_options_exist

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Jun 3, 2025

This PR adds a simple check that all the options mentioned in mathlibStandardSet exist (at least, the ones that do exist!).


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 3, 2025

PR summary 0018ee710e

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

Copy link
Copy Markdown
Contributor

@grunweg grunweg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
maintainer delegate

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 4, 2025

🚀 Pull request has been placed on the maintainer queue by grunweg.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jun 4, 2025
adomani and others added 2 commits June 4, 2025 11:07
@adomani adomani removed the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jun 4, 2025
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Jun 4, 2025

I am maintainer merging a PR that was already maintainer merged, to test the emoji system!

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 4, 2025

🚀 Pull request has been placed on the maintainer queue by adomani.

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jun 4, 2025
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Jun 11, 2025

This PR has been migrated to a fork-based workflow: #25687

@adomani adomani closed this Jun 11, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jun 12, 2025
This PR continues the work from #25407.

Original PR: #25407
pfaffelh pushed a commit to pfaffelh/mathlib4 that referenced this pull request Jun 15, 2025
pfaffelh pushed a commit to pfaffelh/mathlib4 that referenced this pull request Jun 15, 2025
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
@YaelDillies YaelDillies deleted the adomani/check_options_exist branch August 15, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants