Skip to content

Fix Tactics.descend_in_conjunctions on inductives with letins in constructors#21063

Merged
coqbot-app[bot] merged 1 commit intorocq-prover:masterfrom
SkySkimmer:fix-apply-nreal
Sep 22, 2025
Merged

Fix Tactics.descend_in_conjunctions on inductives with letins in constructors#21063
coqbot-app[bot] merged 1 commit intorocq-prover:masterfrom
SkySkimmer:fix-apply-nreal

Conversation

@SkySkimmer
Copy link
Copy Markdown
Contributor

@SkySkimmer SkySkimmer commented Sep 4, 2025

@SkySkimmer SkySkimmer requested a review from a team as a code owner September 4, 2025 15:04
@SkySkimmer SkySkimmer added the request: full CI Use this label when you want your next push to trigger a full CI. label Sep 4, 2025
@coqbot-app coqbot-app Bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Sep 4, 2025
@SkySkimmer SkySkimmer added the needs: overlay This is breaking external developments we track in CI. label Sep 5, 2025
SkySkimmer added a commit to SkySkimmer/VST that referenced this pull request Sep 5, 2025
SkySkimmer added a commit to SkySkimmer/VST that referenced this pull request Sep 5, 2025
…tructors

Fix rocq-prover#21036

Co-authored-by: yannl35133 <yannl35133@users.noreply.github.com>
@SkySkimmer SkySkimmer added request: full CI Use this label when you want your next push to trigger a full CI. and removed needs: overlay This is breaking external developments we track in CI. labels Sep 5, 2025
@coqbot-app coqbot-app Bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Sep 5, 2025
@SkySkimmer
Copy link
Copy Markdown
Contributor Author

VST shows a case where some applys succeed once the bug is fixed (applying an element of a record with a := projection)

@ppedrot ppedrot self-assigned this Sep 12, 2025
andrew-appel pushed a commit to PrincetonUniversity/VST that referenced this pull request Sep 22, 2025
@ppedrot ppedrot added this to the 9.2+rc1 milestone Sep 22, 2025
@ppedrot ppedrot added the kind: fix This fixes a bug or incorrect documentation. label Sep 22, 2025
@ppedrot
Copy link
Copy Markdown
Member

ppedrot commented Sep 22, 2025

@coqbot merge now

@coqbot-app coqbot-app Bot merged commit 3d03dc9 into rocq-prover:master Sep 22, 2025
8 checks passed
@coqbot-app
Copy link
Copy Markdown
Contributor

coqbot-app Bot commented Sep 22, 2025

@ppedrot: Please take care of the following overlays:

  • 21063-SkySkimmer-fix-apply-nreal.sh

@SkySkimmer SkySkimmer deleted the fix-apply-nreal branch September 22, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: fix This fixes a bug or incorrect documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"apply ... in" produces unbound deBruijn

2 participants