The mac test buildbot was broken twice in the last 12 hours because of changes that easily could have been identified by running the new lint tool.
Please try to remember to do this before checking in. I don't think we want to make this a checkin hook, but if it keeps happening, we may need to. On Wed, Dec 10, 2008 at 2:17 PM, Ojan Vafai <[email protected]> wrote: > Firstly, there is now a lint option that will parse the test list for each > platform (win/mac/linux) and each build target (debug/release). This way you > can know that you at least got the syntax right. So, please consider running > this when you make test list changes. It takes ~30 seconds on my machine and > saves you from closing the tree. Also, if you feel moved to improve our > tools a bit, add a pre-commit hook to gcl/git-cl to do this for us > (please!). > $ ./run_webkit_tests.sh --lint-test-files > 081210 11:04:34 run_webkit_tests.py:142 INFO Found: 10244 tests > 081210 11:04:59 run_webkit_tests.py:614 INFO SUCCESS -- Mike Pinkerton Mac Weenie [email protected] --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Chromium-dev" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/chromium-dev?hl=en -~----------~----~----~----~------~----~------~--~---
