4 ms·
I have commit hooks on our repository that look for things like this and prevent the user committing it! We've got 30 odd rules so far that have saved us from
by harrytuttle 13y ago
I have commit hooks on our repository that look for things like this and prevent the user committing it!
We've got 30 odd rules so far that have saved us from all sorts of pain from exception swallowing to adding test ignores as well.
- dave1010uk 13y agoCould you share some of these?
- harrytuttle 13y agoNot really at the moment. I've been slowly refactoring it into a publishable chunk of code but progress is slow. I will publish it on github when it is done.
- mullethunter 13y agoI second this. I manage a couple of teams and one is dealing with legacy a PHP stack where they've been refactoring things like SQL injection risks and whatnot. Having something as an automated second set of eyes would be brilliant.
- tomjen3 13y agoI am not sure that is enough, I mean if you look at the supposed results many of them can't actually be actacked (like the on that tests that Id is a number before it is used) but a semi stupid script wouldn't catch those.
- harrytuttle 13y agoIt's good enough to make people aware that you are watching their every move :)