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
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 |