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

Metadata is not passed on form evaluation #1551

Closed
daslu opened this issue Feb 20, 2022 · 2 comments · Fixed by #1566
Closed

Metadata is not passed on form evaluation #1551

daslu opened this issue Feb 20, 2022 · 2 comments · Fixed by #1566

Comments

@daslu
Copy link

daslu commented Feb 20, 2022

Just noticed a little gap between CIDER and Calva.
If we have the following code:

^:abcd [1]

(that is, a vector with a number in it, with some metadata before it), and we put the cursor after this piece of code,
then cider-eval-last-sexp would pass the code ^:abcd [1] to nREPL,
while Calva: Evaluate Current Form would pass only [1].

@PEZ
Copy link
Collaborator

PEZ commented Feb 20, 2022

Thanks. I agree with CIDER about this. We treat reader tags as part of the form they tag, and this is very similar.

What do we say about ignores? They are reader tags, but treated a bit differently (not part of the form they ignore out) by Calva, to make it convenient to evaluate an ignored form. What does CIDER do here?

@PEZ PEZ added the evaluation label Feb 20, 2022
@PEZ PEZ linked a pull request Mar 1, 2022 that will close this issue
14 tasks
@PEZ
Copy link
Collaborator

PEZ commented Mar 1, 2022

Here's a VSIX fixing most of this issue: https://18486-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.247-1551-current-form-metadata-58199575.vsix

Please put it to some tests. See #1566 about what to expect.

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

Successfully merging a pull request may close this issue.

2 participants