[Qgis-developer] Scrutinizer testing now enabled

[Nyall Dawson]

On 14 August 2015 at 15:36, Nyall Dawson <nyall.dawson at gmail.com> wrote:
> Hey all,
>
> Following up the recent launch of CI testing under OSX, Tim has now
> enabled Scrutinizer checks on the QGIS master branch.
> What this means is that now every commit & PR is tested by
> Scrutinizer's checker. This checks for bugs and bad practices in
> Python code and will alert us of any potential issues prior to merging
> PRs. You can see the results of each check at the bottom of recent PRs
> - you should see both the results of this check and Travis' Linux/OSX
> testing.

Unfortunately this hasn't tested well.... the results seem quite
random (reporting changes on files which haven't been touched) and
I've been unsuccessful with contacting Scrutinizer's support to figure
out why.

Tim (or someone else with admin rights to the git repo) - can you
remove this from the QGIS repo for now? We can re-enable if
Scrutinizer are able to fix these issues....

Nyall


>
> It'll be interesting to see how useful this is in the long run, but in
> any case it's a free service (for open source projects) and doesn't
> require any extra infrastructure on our behalf. Hopefully it's an easy
> win and is able to prevent some Python errors slipping in to the code
> in future....
>
> Nyall