couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ilya Khlopotov <>
Subject Re: Reformat src files with `erlfmt` on `main`
Date Thu, 20 May 2021 15:53:16 GMT
Good idea Donat!!! 

Even though I disagree with some of the choices made by erlfmt I appreciate consistency it
The choices are logical. I really love that every decision is documented and properly discussed.
I did read PR in its entirety and in fact was not even noticed the ugly `->` in the beginning
of the line closer to the end of the review process. 
I do believe our wetware would adjust in no time to new formatting. Given how easy it is to
reason about. I agree with Donat's observation that we are spending too much time and emphasis
on formatting issues every time we review PRs. I do believe it is a machine job to provide
consistent formatting. We humans are better at other things. All in all I vote for adopting
`erlfmt` for both 3.x and main.

Also thank you Donat for providing validation scripts to make sure the re-formatted code compiles
to the same beam files.

Best regards,

On 2021/05/18 18:13:14, Bessenyei Balázs Donát <> wrote: 
> Hi dev@couchdb,
> To eliminate the need for formatting-related comments and thus
> unnecessary cycles in PRs, I've invested a little time to see if we
> could use a formatter on `main` [1].
> The PR reformats `.erl` files in `src` and the script [2] included
> shows that the compiled binaries match "before" and "after".
> The formatter used in the PR is `erlfmt` [3] which is an opinionated
> [4] tool so it's more of a "take it or leave it" as-is. (We could try
> using other formatters if we want in case people want formatting but
> think the choices `erlfmt` makes are unacceptable.)
> Some members of the CouchDB dev community already left some great
> comments on the PR and I haven't seen any strong opposition so far,
> but I wanted to make sure more people are aware of this.
> If you have any questions, comments or concerns (or objections),
> please let me know.
> Thank you,
> Donat
> [1]:
> [2]:
> [3]:
> [4]:

View raw message