Last modified: 2010-07-26 17:11:04 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T26543, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 24543 - Commit hook for syntax checking
Commit hook for syntax checking
Status: RESOLVED DUPLICATE of bug 20069
Product: Wikimedia
Classification: Unclassified
Subversion (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Rob Halsell
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-26 07:11 UTC by Sam Reed (reedy)
Modified: 2010-07-26 17:11 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2010-07-26 07:11:49 UTC
Many of us have probably done this at some point or another - Where we've committed without pressing save again, or a typo in a variable.

Meaning we need a followup commit

r69921 and then r69930 to fix it, for example

Any chance of a hook to just literally do a syntax/sanity check of the commited code, aborting the commit if it doesn't check out?
Comment 1 Max Semenik 2010-07-26 13:25:43 UTC
For the reference: there is maintenace/checkSyntax.php which checks not only for syntax errors, but also for other common problems, such as BOM's. When run on working copy with --modified switch, it will check quickly only modified files. It's mentioned at <http://www.mediawiki.org/wiki/Manual:Pre-commit_checklist>, but that page is not advertised enough.
Comment 2 Chad H. 2010-07-26 16:59:38 UTC
Pre-commit? Ugh...

For the same reasons as bug 21211, this would be annoying. Not opposed to the idea in concept, but it needs some fleshing out.

Perhaps if we can come up with a non-blocking way to do hooks first.
Comment 3 Max Semenik 2010-07-26 17:11:04 UTC

*** This bug has been marked as a duplicate of bug 20069 ***

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links