Interpreter can't reduce top-level definitions #1247
Assignees
Labels
bug 🐞
A confirmed bug
primer
Specific to the primer package
priority: high
This issue has high priority
Comments
dhess
added
priority: high
dhess
added a commit
that referenced
this issue
Apr 22, 2024
Note that all of these tests that use the interpreter are currently expected to fail (or are commented out because they will fail) due to #1247 Signed-off-by: Drew Hess <[email protected]>
dhess
added a commit
to hackworthltd/primer-app
that referenced
this issue
Apr 23, 2024
This is more or less a drop-in replacement for `EvalFull` and is ready to go, save for hackworthltd/primer#1247. Therefore, we don't yet use it in the `Edit` and `PictureInPicture` components. Signed-off-by: Drew Hess <[email protected]>
This was referenced Apr 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code of conduct
Don't report security issues here
Is there an existing issue?
What happened?
While adding the interpreter to the various Primer APIs, I discovered that the interpreter can't reduce top-level definitions. This test demonstrates that the interpreter can't evaluate the
even 3?top-level definition in our exampleeven3Prog:What should have happened?
The interpreter should be able to reduce a top-level definition whose root node is an
App.Which package(s) or aspect(s) of the project are the source of the issue?
primer
Developers, please provide the output of the bug report script
No response
Do you have a reproducer?
Yes, see above.
Any additional information?
We can't use the interpreter in the frontend until this issue is resolved. See hackworthltd/primer-app#1169
The text was updated successfully, but these errors were encountered: