-
Couldn't load subscription status.
- Fork 44
eval: add --cue option #255
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
Conversation
For testing the behavior of the command line application itself, I think the best way to do this is by writing a shell script that invokes |
|
When @joemfb removes his changes requested we'll reconsider merging this. For now, looks like this is still under deliberation. |
bc829b8 to
652c573
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will do (with a caveat about the newt usage pattern)
Partially Resolves #197
Adds the
--cueoption to theevalutility. This allows jammed nouns to be passed overstdinand pretty-printed tostdout.Also includes a sub-option which will expect the incoming jammed noun to be Khan output. If the thread resulted in a
goof, it's pretty-printed tostderras a stack trace and the output is simply theconn.cheader info (unique ID,%avowmark signalling Khan output,%.nloobean signalling thread failure).Also includes the "todo"s from the comments for #155.
Testing:
Not strictly necessary for this PR, but I would like to add explicit testing for both
--jamand--cue. @mcevoypeter @matthew-levan If either of you can recommend how to modify Bazel to do so, it would be appreciated.