Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: structural nested recursion confused when nested type appears first #5766

Merged
merged 2 commits into from
Oct 18, 2024
Merged

fix: structural nested recursion confused when nested type appears first #5766

merged 2 commits into from
Oct 18, 2024

Conversation

nomeata
Copy link
Collaborator

@nomeata nomeata commented Oct 18, 2024

this fixes #5726

@nomeata nomeata added the will-merge-soon …unless someone speaks up label Oct 18, 2024
@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 18, 2024
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Oct 18, 2024
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#5766
1edd70d
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Oct 18, 2024
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#5766
bb2f22b
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Oct 18, 2024 •
edited
Loading

Mathlib CI status (docs):

  • ✅ Mathlib branch lean-pr-testing-5766 has successfully built against this PR. (2024-10-18 15:50:25) View Log
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 11ae8bae42509eaca43db69ed95fbdd9570d5039 --onto 3a34a8e0d119d8a907614af62b405d85149672f0. (2024-10-18 16:20:21)

@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Oct 18, 2024
@nomeata nomeata enabled auto-merge October 18, 2024 16:05
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc October 18, 2024 16:17 Inactive
@nomeata nomeata added this pull request to the merge queue Oct 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 18, 2024
@nomeata nomeata added this pull request to the merge queue Oct 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 18, 2024
@nomeata nomeata added this pull request to the merge queue Oct 18, 2024
Merged via the queue into master with commit 26df545 Oct 18, 2024
14 checks passed
@nomeata nomeata deleted the joachim/issue5726 branch October 25, 2024 14:40
tobiasgrosser pushed a commit to opencompl/lean4 that referenced this pull request Oct 27, 2024
@nomeata @tobiasgrosser
fix: structural nested recursion confused when nested type appears fi…
fa48902 
…rst (leanprover#5766)

this fixes leanprover#5726
@nomeata nomeata removed the will-merge-soon …unless someone speaks up label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
builds-mathlib CI has verified that Mathlib builds against this PR toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

PANIC at outOfBounds on theorem involving a nested inductive
2 participants
@nomeata @leanprover-community-bot