Clojurians Log v2
parinfer
added some early thoughts for trying parinfer in Bruce’s new readline library: https://github.com/bhauman/rebel-readline/issues/84
parinfer-jvm will have to be updated
which is cool, because Timothy’s parinfer-based code validator will need it too: https://github.com/timothypratley/parsnip
The reaction to parinfer smart mode in Cursive can be fairly described as “rapturous”
ha, wow
you getting this feedback in the #cursive channel or elsewhere?
A bit by email, some here (mostly DMs), some on Twitter
But people really seem to like it.
And the more concise: https://twitter.com/solussd/status/961808691220615168
etc etc
@shaunlebron Did you get a chance to play around with the incorrect indentation marking? Any opinions on that?
Nice feedback! I wonder how many are trying it out. What would make you comfortable deploying it on next release?
Pulling up Cursive now to play with the markers. My only issue with it in theory is that wrong indentation anywhere in the file will prevent Parinfer from working everywhere in the file
Right, that’s the one thing I’d really like to fix. The user should be able to fix it themselves by just reformatting the file, of course.
And I like that they have to do it manually - that means that their files don’t get reformatted just by looking at them.
I’m going to investigate the “apply per top level form” change. If I can get that working then this will definitely become the next release right away. But honestly even without that this is still a vast improvement over what’s there anyway.
This is what turned some of my coworkers off from using Parinfer originally. They had a lot of poorly formatted/indented files and just opening them would rearrange and sometimes break things.
And what's the plan for atom-parinfer in terms of this new release? (I'm not even sure who is responsible for cutting releases of that)
Yeah, this has annoyed me too. In a lot of my tests, I used to leave a dangling close paren at the end of my deftests, because it made it easier to copy assertions per line.
Ironically of course, parinfer makes that more or less obsolete, but it’s still annoying to have a bunch of changes every time I look at a test file.
@taylor: it’s one thing for it to annoy, but yeah, it should never break. that was a problem with a race condition in intellij I believe
Right, breaking is a bug, the reformatting is just a bit annoying.
FWIW this was like over a year ago, not sure what has changed in main-line Cursive+Parinfer since
I’ve been happily using Cursive+Parinfer the whole time 🙂
Essentially nothing, sadly.
I’ll leave my feedback in #cursive
@seancorfield i’m maintaining atom-parinfer for smart mode things, but chris is still the gatekeeper on releases
feedback on atom-parinfer has been silent, except for your feedback sean. I still need to fix this: https://github.com/oakmac/atom-parinfer/issues/100
Okay. Thanks for the clarification on releases.