Skip to content

Make compatible with upcoming version of Lamdera#650

Merged
lydell merged 2 commits intortfeldman:masterfrom
miniBill:master
Jul 6, 2025
Merged

Make compatible with upcoming version of Lamdera#650
lydell merged 2 commits intortfeldman:masterfrom
miniBill:master

Conversation

@miniBill
Copy link
Contributor

@miniBill miniBill commented Jul 6, 2025

No description provided.

@lydell
Copy link
Collaborator

lydell commented Jul 6, 2025

Awesome!

Having read lamdera/compiler#56 before, I was imagining the diff to be something like F2\( turned into (F2\()? – but nope, it’s an added \$?! So Lamdera is now adding an extra $ to variable names? Just so that I understand the whole thing, which change in Lamdera is that from? And out of curiosity – why?

@miniBill
Copy link
Contributor Author

miniBill commented Jul 6, 2025

The change is from the Fx/Ax patch. The added $ is the "unwrapped" version, whereas the version without is the original Fx-wrapped one.

@lydell lydell merged commit 2ae97cc into rtfeldman:master Jul 6, 2025
20 checks passed
@lydell
Copy link
Collaborator

lydell commented Jul 6, 2025

Thanks! Released in 0.19.1-revision16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants