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.

Sign up for GitHub

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

Jump to bottom

Can't abandon multiline sequence #212

Closed
marcelocantos opened this issue May 9, 2020 · 0 comments · Fixed by #217
Closed

Can't abandon multiline sequence #212

marcelocantos opened this issue May 9, 2020 · 0 comments · Fixed by #217
Labels
bug P0 High priority issue

Comments

@marcelocantos
Copy link
Contributor

marcelocantos commented May 9, 2020 •
edited
Loading

Please do not post any internal, closed source snippets on this public issue tracker!

Description

If you accidentally enter a syntax error halfway through a multiline sequence, you can't
escape out of it. It should be possible to abandon the sequence with Ctrl+C.

Steps to Reproduce

  1. Try the following: 
    @> (x +<Enter>
 > <Ctrl+C>

Expected behavior:

Exits back to @> prompt:

@> (x +<Enter>
 > <Ctrl+C>
@>

Actual behavior:

Repeats continuation sequence:

@> (x +<Enter>
 > <Ctrl+C>
 > <Ctrl+C>
 > <Ctrl+C>
 > <Ctrl+C>
 >
@marcelocantos marcelocantos added P0 High priority issue bug labels May 9, 2020
@marcelocantos marcelocantos mentioned this issue May 9, 2020
Merged
anzopensource pushed a commit that referenced this issue Jul 9, 2021
@nofun97
import cache resets everytime EvaluateExpr is called (#212)
a4e26a2 
cache was implemented as a global variable and when arrai code is called from golang,
the import cache persists. This commit moves import cache to context and ensures it
is reset every time arrai code is called in a golang code. 

It also allows `EvaluateExpr` to receive a context argument with existing cache inside. The function will not override this cache so that cache can be reused. This is meant for those who run arrai scripts in a golang program.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug P0 High priority issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ctrl-C cancels multiline input arr-ai/arrai
1 participant
@marcelocantos