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

Miller script #177

Closed
derekmahar opened this issue May 25, 2018 · 3 comments
Closed

Miller script #177

derekmahar opened this issue May 25, 2018 · 3 comments

Comments

@derekmahar
Copy link
Contributor

derekmahar commented May 25, 2018

Add an option to process a script file containing a sequence of verbs similar to what then-chaining achieves now by chaining together verbs.

@johnkerl
Copy link
Owner

johnkerl commented Jun 7, 2018

This is trivial & very easy to do except for if the then-chain includes a put or filter in which case the parsing needs to be a bit complicated ... honestly I don't see a huge advantage over shell scripts but as long as I enforce rules about put/filter expressions being single-quoted, and correctly handle embedded-but-escaped single-quotes within the put/filter expression ... it should be doable.

It feels like reinventing a fair amount of shell-script parsing logic which makes me think shell-scripting is already the way to go though ...

@derekmahar
Copy link
Contributor Author

Yes, perhaps you're right. I was thinking that conditional evaluation might be useful while minimizing the mlr invocations, but existing shell scripts and then-caching serve this purpose well enough already.

@johnkerl
Copy link
Owner

johnkerl commented Sep 7, 2019

OK, thank you! :)

@johnkerl johnkerl closed this as completed Sep 7, 2019
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

2 participants