jsxer icon indicating copy to clipboard operation
jsxer copied to clipboard

Published 20 hours ago • verified publisher icon AngeloD2022

added indented ast-node-printer

Open psyirius opened this issue 1 year ago • 12 comments

Addresses: #42

psyirius avatar Mar 28 '23 14:03 psyirius

awesome, I'll check this out later today!

AngeloD2022 avatar Mar 28 '23 14:03 AngeloD2022

Nice work. Can we add a CLI flag that enables this?

AngeloD2022 avatar Mar 28 '23 22:03 AngeloD2022

Nice work. Can we add a CLI flag that enables this?

Yeah. I'm integrating a new cli parser to make the cli easier to use. We can roll it out with that change.

BTW, if you got the reference file for #82 mail me.

psyirius avatar Mar 29 '23 02:03 psyirius

I'll get those files to you asap

AngeloD2022 avatar Mar 29 '23 03:03 AngeloD2022

Nice work. Can we add a CLI flag that enables this?

DONE!

image

image

psyirius avatar Mar 29 '23 04:03 psyirius

awesome! I'll check this out!

AngeloD2022 avatar Mar 29 '23 04:03 AngeloD2022

So I did a checkout and it doesn't give me the --ast or --verbose options. did you commit those yet?

AngeloD2022 avatar Mar 29 '23 04:03 AngeloD2022

So I did a checkout and it doesn't give me the --ast or --verbose options. did you commit those yet?

yes. lemme double check it

psyirius avatar Mar 29 '23 06:03 psyirius

Yes i am.

psyirius avatar Mar 29 '23 06:03 psyirius

I'm an idiot and forgot to git pull haha. my bad

AngeloD2022 avatar Mar 29 '23 06:03 AngeloD2022

I'm an idiot and forgot to git pull haha. my bad

pull again. pushed some more changes.

psyirius avatar Mar 29 '23 06:03 psyirius

Looks like there is a circular dependency between jsxer.h/cpp (depends on Reader from reader.h/cpp, which depends on Options from jsxer.h/cpp)

AngeloD2022 avatar May 24 '23 05:05 AngeloD2022