search

dundalek / closh

Bash-like shell based on Clojure
Eclipse Public License 1.0
1.61k stars 65 forks source link

let sci do the parsing #161

Closed borkdude closed 4 years ago

borkdude commented 4 years ago

This now works:

$ ./closh-zero-sci 'ls |> (filter #(re-find #"C" %))'
(CHANGELOG.md LICENSE)%

NOTE: we're relying on sci interpreter implementation details here :).

dundalek commented 4 years ago

Cool approach :+1:

Unfortunately when I run tests locally it increases number of failures. There are also few StackOverflowError errors, so maybe that's what is causing GH Actions runner to hang.

I will need to investigate deeper to figure out what is going on.

dundalek commented 4 years ago

I ended up using different approach with reader customization, but this was very helpful showing how to hook into sci interpreter internals. Thank you, I will close this now.