3 ms·
Extract expression is such a common refactoring, happy to see support for it. I'm even more curious to see now if there are going to be more advanced refactorin
by mhitza 1y ago
Extract expression is such a common refactoring, happy to see support for it. I'm even more curious to see now if there are going to be more advanced refactoring possible. A List.map, to for loop, and back refactoring would be such a great thing to have in how I program.
Only thing that would go on my OCD nerve, is the lack of an empty newline when the show_markup function is extracted. Kind of "common sense" when writing top level bindings to leave some breathing room between them.
- nukifw 1y agoFor the first point, you are right. We start by a common (but very useful) feature. Since OCaml allows an infinite level of nesting (and different kind of structure item) it was still a bit of a challenge, mostly for finding the right UX. For the second point we delay the aeration convention to the formatter (ocamlformat). It can be configure in a different way :) Thanks for your feedback!