janet-lang / jpm

Janet Project Manager
MIT License
65 stars 21 forks source link

Add chance to eval code not only on top level #57

Closed pepe closed 1 year ago

pepe commented 1 year ago

I am wrangling with the idents in this PR. I recommend using the spork/format in the official janet-lang repositories. I will reindent it to its old form if it is an issue.

pepe commented 1 year ago

I was thinking about the setfn macro, and one solution I see fit is to add the set- prefix to the indent-2-peg of the spork/fmt and rename the function accordingly. What do you think about such a solution?

bakpakin commented 1 year ago

LGTM - original idea was to rely on Janet level macros only for non-top level preprocessing, but there is no reason to not allow this.