New subject: [Phpmyadmin-devel] Changes in SOB checking in pull requests

20 Mar 2014

On 3/19/14 2:09 PM, Michal Čihař wrote:
...
  Hi

 Dne Thu, 13 Mar 2014 11:54:25 +0100
 Michal Čihař <michal@...> napsal(a):

  Source code for the hook is in our scripts repo:

 https://github.com/phpmyadmin/scripts/tree/master/hooks

 This has advantage that authors get direct feedback mentioning what is
 wrong and also these comments are directly visibly in pull request. 
 I've just added check for tab characters in the code to the same hook,
 the sample message it has fired can be seen here:

https://github.com/ganeshaditya1/phpmyadmin/commit/396ac695d439b0ac37f9c7c7…

Great work Michal. These improvements are the "little things" that make
development and code checks so much easier over time. It's great to see
these checks improve our tests and coverage.

Hey,

I am using sublime editor to write my code and after I got that error
message I checked to see if I am using tabs for indentation and turns out
that I am not using tabs for indentation and also just to be sure I
selected all my source code and used this option in sublime that
automatically replaces tabs with 4 spaces and re-uploaded the patch.

Could you please check it out once, seems to me that there is some bug in
this system at your end.

Regards,
Aditya

Re: [Phpmyadmin-devel] Changes in SOB checking in pull requests