Skip to content

Conversation

@oskgo
Copy link
Contributor

@oskgo oskgo commented Dec 9, 2025

Fixes #847

@oskgo
Copy link
Contributor Author

oskgo commented Dec 9, 2025

@mbbarbosa See if this PR fixes the example you used to come up with #847.

@fdupress fdupress requested review from mbbarbosa and strub December 9, 2025 13:25
@fdupress fdupress assigned fdupress and oskgo and unassigned fdupress Dec 9, 2025
@fdupress fdupress added the bug label Dec 9, 2025
@mbbarbosa
Copy link
Contributor

mbbarbosa commented Dec 10, 2025

I did a very blunt merge of this fix into the branch I am working in and all my problems are gone. :-)
So hopefully things will indeed work out once we go via the proper merge sequence.
Thanks!

@strub
Copy link
Member

strub commented Dec 18, 2025

@oskgo csn you squash and merge?

@oskgo oskgo merged commit 9f8d176 into main Dec 18, 2025
15 checks passed
@oskgo oskgo deleted the fix-847 branch December 18, 2025 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle conseq when the combined statement has a non-empty ambient logic context.

5 participants