ok to do reformatting commits?

Simon Peyton Jones simonpj at microsoft.com
Wed Nov 25 13:03:59 UTC 2015


I don’t have a strong view but like Richard I find 80 chars limiting.

I agree that we should resolve this and write down the result on the coding style page

Simon

|  -----Original Message-----
|  From: ghc-devs [mailto:ghc-devs-bounces at haskell.org] On Behalf Of
|  Richard Eisenberg
|  Sent: 25 November 2015 03:23
|  To: Evan Laforge <qdunkan at gmail.com>
|  Cc: ghc-devs at haskell.org
|  Subject: Re: ok to do reformatting commits?
|  
|  Thanks for volunteering to do this work, but I'm afraid now is a
|  terrible time to do it. I know of three significant patches that are
|  about to be committed, and your reformatting would cause quite a few
|  merge conflicts. If there is a lull between a feature freeze and a
|  ghc-8.0 fork, that would be the ideal time, to my mind.
|  
|  That said, I remain unconvinced that a rigid commitment to 80-char
|  lines is in our best interest. My personal vote is to continue to have
|  80 characters as a guideline but to keep the current practice of
|  allowing programmer discretion.
|  
|  Richard
|  
|  On Nov 24, 2015, at 10:14 PM, Evan Laforge <qdunkan at gmail.com> wrote:
|  
|  > When I was doing a recent patch, I was annoyed by lint errors about
|  >> 80 lines when I was just conforming to the existing style.  To
|  avoid
|  > cluttering my commit with unrelated changes, I decided to fix the
|  > lints in a formatting-only commit afterwards.  Looking in the
|  > archives, I see there was some recent discussion about this, but I
|  > didn't see anyone volunteering to just go wrap a bunch of files, or
|  > saying that they didn't want anyone to do this (usual reason being
|  > cluttering the history, which as a rationale to not do formatting
|  only
|  > changes never sat too well with me).
|  >
|  > Would anyone mind if I went and wrapped a bunch of files, say
|  > typecheck/*.hs?  This seems simpler than either constant hassling
|  from
|  > arc or coming up with more elaborate rules for arc.  I would have to
|  > make some formatting decisions, so likely to some eyes I would be
|  > messing some stuff up, but since there's no real standard that is
|  > probably unavoidable.
|  > _______________________________________________
|  > ghc-devs mailing list
|  > ghc-devs at haskell.org
|  >
|  https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fmail.h
|  > askell.org%2fcgi-bin%2fmailman%2flistinfo%2fghc-
|  devs&data=01%7c01%7csi
|  >
|  monpj%40064d.mgd.microsoft.com%7c27ba726d65bd49df735d08d2f547c211%7c72
|  >
|  f988bf86f141af91ab2d7cd011db47%7c1&sdata=iGQx%2bYYoG%2bv7xCd6Su%2bzN1L
|  > gIjx5FxEqmWOSpIbLnjY%3d
|  
|  _______________________________________________
|  ghc-devs mailing list
|  ghc-devs at haskell.org
|  https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fmail.h
|  askell.org%2fcgi-bin%2fmailman%2flistinfo%2fghc-
|  devs&data=01%7c01%7csimonpj%40064d.mgd.microsoft.com%7c27ba726d65bd49d
|  f735d08d2f547c211%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=iGQx%2b
|  YYoG%2bv7xCd6Su%2bzN1LgIjx5FxEqmWOSpIbLnjY%3d


More information about the ghc-devs mailing list