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.

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

Escaping glue's brackets ("{") #27

Closed
wants to merge 1 commit into from

Conversation

notPlancha
Copy link

@notPlancha notPlancha commented Feb 3, 2024

Fixes #26
I quickly tested and it passes the tests so I thought I might as well

@lewinfox
Copy link
Owner

lewinfox commented Feb 4, 2024

Thanks for raising this. It definitely seems like a bug - I'll take a look at the PR over the next few days 👍👍

@lewinfox lewinfox self-requested a review February 17, 2024 09:37
@lewinfox lewinfox self-assigned this Feb 17, 2024
@lewinfox
Copy link
Owner

@notPlancha thanks for the PR. I'm not keen on always performing the find-and-replace because I'm sure at some point that will break someone else's workflow and we will have another issue raised.

Instead, I've opted to pre-validate the various bits of glue code, and if any errors are raised I apply your solution. You can see the details here: #28

@lewinfox lewinfox closed this Feb 17, 2024
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.

"{}" causes ic() to break
2 participants