What about “MinimizeLines” since it tries to minimize the number of lines? Or maybe “MinimumLines”.
axelson
Scenic Core Team
Marcus
I like those names. So that I do not have to decide I would suggest MinLines. It would also be a little shorter. WDYT?
0
dimitarvp
Not sure “minimizing lines” is what this does. I’d call it just EnforceLineLength or some such. Keeping in mind that this should be the default behaviour of any formatter anyhow (which apparently is an opinion and not an accepted fact. 
).
Or maybe ConservativeWhitespace… 
1
tensiondriven
SameLine isn’t bad – What comes to mind for me is UnwrapLines - this seems an apt name since we are removing wrapping.
I’m excited to try this out – I tried adding recode to a big project, but it didn’t seem to register the mix tasks for some reason. I tried in a new mix project and it worked, though. (An option to pass it a directory or file would be rad - or maybe the ability to use it from the cli outside of project context - either of those would solve my immediate problem.)
1
tensiondriven
Update: I had the dep configured wrong, after fixing it works like a champ! Thank you Marcus, this is amazing.
0
dimitarvp
axelson
Scenic Core Team
Another naming idea is PreferSameLine since getting all the code on the same online is not always possible.
2
dimitarvp
Marcus
Thanks for all your suggestions. I have already renamed it to `EnforceLineLength". I hope that is fine for all.
1