pgindent-polluted commits

From: Noah Misch <noah(at)leadboat(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: pgindent-polluted commits
Date: 2016-01-13 14:48:26
Message-ID: 20160113144826.GB3379802@tornado.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I've noticed commits, from a few of you, carrying pgindent changes to lines
the patch would not otherwise change. (That is to say, the next pgindent run
would have made the same changes anyway.) From
https://wiki.postgresql.org/wiki/Submitting_a_Patch#Reasons_your_patch_might_be_returned:

The fastest way to get your patch rejected is to make unrelated changes.
Reformatting lines that haven't changed, changing unrelated comments you
felt were poorly worded, touching code not necessary to your change, etc.

Commits should follow the same high standards we ask of submissions. Several
pgindent strategies do conform:

1) Run pgindent, then manually reduce its changes to the subset that your
patch caused.
2) Don't run pgindent yourself; commit code that pgindent may later change.
3) Push a commit containing nothing but a pgindent run of the files you care
about, then push a second commit for your feature/bugfix.

Please use of one of those next time you'd run pgindent.

Thanks,
nm

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Vladimir Sitnikov 2016-01-13 14:49:32 Re: Fwd: [JDBC] Re: 9.4-1207 behaves differently with server side prepared statements compared to 9.2-1102
Previous Message Andres Freund 2016-01-13 14:43:18 Re: Fwd: [JDBC] Re: 9.4-1207 behaves differently with server side prepared statements compared to 9.2-1102