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

Issue when strings containing @ are simplified #451

Closed
alexaandru opened this issue May 27, 2023 · 3 comments
Closed

Issue when strings containing @ are simplified #451

alexaandru opened this issue May 27, 2023 · 3 comments

Comments

@alexaandru
Copy link
Contributor

First of all, apologies if this is not the right place for reporting bugs related to fnlfmt. I would appreciate if you pointed me to the right place if that's the case.

Now for the actual issue:

Given an x.fnl file containing:

{"@foo.bar" 42}

fnlfmt x.fnl produces {:@foo.bar 42} which then throws x.fnl:1:2 Parse error: invalid character: @ error.
I use Fennel 1.3.1-dev on PUC Lua 5.4 and fnlfmt version 0.3.1-dev.

@technomancy
Copy link
Collaborator

No problem; this is fine, thanks. In fact, this is a bug in Fennel itself; you get the same problem putting that table into the Fennel repl outside antifennel. The root cause is probably in src/fennel/view.fnl.

@technomancy technomancy changed the title [fnlfmt] Issue when strings containing @ are simplified Issue when strings containing @ are simplified May 27, 2023
@technomancy
Copy link
Collaborator

This should be fixed in 8404556; thanks for reporting.

@alexaandru
Copy link
Contributor Author

Awesome & you're most welcome! Cheers! :-)

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

No branches or pull requests

2 participants