http://www.102degrees.com/blog/2008/09/21/svn-pre-commit-duty-lint-your-php/ Web Programming and Design by Aaron Saray Tue, 06 Jan 2009 03:51:34 +0000 http://wordpress.org/?v=2.7 hourly 1 http://www.102degrees.com/blog/2008/09/21/svn-pre-commit-duty-lint-your-php/comment-page-1/#comment-762 Travis Mon, 22 Sep 2008 21:44:36 +0000 http://www.102degrees.com/blog/?p=225#comment-762 Good point, Aaron, hadn't thought of that. Guess I'll have to actually work instead of being that lazy. :-P Good point, Aaron, hadn’t thought of that. Guess I’ll have to actually work instead of being that lazy.

]]> http://www.102degrees.com/blog/2008/09/21/svn-pre-commit-duty-lint-your-php/comment-page-1/#comment-761 Aaron Mon, 22 Sep 2008 18:03:06 +0000 http://www.102degrees.com/blog/?p=225#comment-761 @Travis: I thought about this - but SVN Stat can generate output for modified folders - and PHP will then start spitting out all these messages like "Cannot open input file". I did not want those to pop up. Good points on adding in new file extensions - you'll just have to remember what they are and modify the script! @Travis: I thought about this - but SVN Stat can generate output for modified folders - and PHP will then start spitting out all these messages like “Cannot open input file”. I did not want those to pop up. Good points on adding in new file extensions - you’ll just have to remember what they are and modify the script!

]]> http://www.102degrees.com/blog/2008/09/21/svn-pre-commit-duty-lint-your-php/comment-page-1/#comment-760 Travis Mon, 22 Sep 2008 17:25:38 +0000 http://www.102degrees.com/blog/?p=225#comment-760 Wouldn't it also work to simply grab *all* files from the output of svn stat and run those through the linting process? Unless I'm terribly mistaken, the PHP parser will essentially ignore everything that isn't between tags (or if you have short tags enabled) - this way, if you get crazy and start doing .pjs or .pcss files, or if you happen to have code where included files have a .inc extension, or any other such ridiculousness, your lint script still works, and it shouldn't really take all that much extra time for PHP to say "Oh look, there's no PHP in this file, moving on!" Wouldn’t it also work to simply grab *all* files from the output of svn stat and run those through the linting process? Unless I’m terribly mistaken, the PHP parser will essentially ignore everything that isn’t between tags (or if you have short tags enabled) - this way, if you get crazy and start doing .pjs or .pcss files, or if you happen to have code where included files have a .inc extension, or any other such ridiculousness, your lint script still works, and it shouldn’t really take all that much extra time for PHP to say “Oh look, there’s no PHP in this file, moving on!”

]]>