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

Add a test for nested unquotes #568

Open
masak opened this issue Sep 18, 2021 · 1 comment
Open

Add a test for nested unquotes #568

masak opened this issue Sep 18, 2021 · 1 comment

Comments

@masak
Copy link
Owner

masak commented Sep 18, 2021

Just realized in #567 that unquote (the alternative {{{ }}} syntax) supports nested unquotes, and that we don't have test coverage for that at all. We should add at least one test for that.

Yes, that will mean implementing the unquote syntax in the first place, one way or another. It's high time we did that.

@masak
Copy link
Owner Author

masak commented Jan 30, 2023

As part of writing such a test, we also need to decide what the actual semantics of nested unquoting should be. I think the two main candidates are:

  • Interpolate N levels out (as in Lisp quasiquotes)
  • Disallow it with a compile-time error

And I don't think this behavior is a contender:

  • Just pretend we didn't see the inner quote, and interpolate 1 level out (either failing, or succeeding in some unlikely way that might as well have been a failure)

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

1 participant