Skip to content

feat: record all structure parents in StructureInfo#5853

Merged
kmill merged 1 commit intoleanprover:masterfrom
kmill:record_parents
Oct 28, 2024
Merged

feat: record all structure parents in StructureInfo#5853
kmill merged 1 commit intoleanprover:masterfrom
kmill:record_parents

Conversation

@kmill
Copy link
Copy Markdown
Collaborator

@kmill kmill commented Oct 26, 2024

Followup to #5841. Makes the structure command populate the new parentInfo field with all the structures in the extends clause.

This will require a stage0 update to fully take effect.

Breaking change: now it's a warning if a structure extends a parent multiple times.

Breaking change: now getParentStructures is getStructureSubobjects. Adds getStructureParentInfo for getting all the immediate parents. Note that the set of subobjects is neither a subset nor a superset of the immediate parents.

Closes #1881

@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Oct 26, 2024
@ghost
Copy link
Copy Markdown

ghost commented Oct 26, 2024

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 8b5443eb22252c9982c23e8fd15eb67d902da65d --onto 193b6f2bec332ac0bce33e10856a96163d4be456. (2024-10-26 21:14:00)
  • 💥 Mathlib branch lean-pr-testing-5853 build failed against this PR. (2024-10-28 00:37:22) View Log

Followup to leanprover#5841. Makes the `structure` command populate the new `parentInfo` field.

This will require a stage0 update to fully take effect.

Breaking change: now `getParentStructures` is `getStructureSubobjects`. Adds `getStructureParentInfo` for getting all the immediate parents. Note that the set of subobjects is neither a subset nor a superset of the immediate parents.

Closes leanprover#1881
ghost pushed a commit to leanprover-community/batteries that referenced this pull request Oct 28, 2024
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Oct 28, 2024
@ghost ghost added the breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan label Oct 28, 2024
@kmill kmill added this pull request to the merge queue Oct 28, 2024
Merged via the queue into leanprover:master with commit 9847923 Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFC] record all parents structures in StructureInfo

1 participant