-
Notifications
You must be signed in to change notification settings - Fork 648
quantifiedcode
quantifiedcode is a tool that checks code for best practices and finds bugs. It also advertises that it can send PR to a github project that correct issues and it can supposedly run checks on PRs.
It is now enabled for MDAnalysis (but the badge above only points to the master branch whereas we really care about develop):
https://www.quantifiedcode.com/app/project/829d7d4b95d04111b38451d14101545d?branch=origin%2Fdevelop
We can use quantifiedcode in the same way as one uses the spell checker in a text processor: fix the obvious mistakes and use most of it as suggestions.
Most issues are not auto-fixable and in any case, even magic PR don't seem to work at the moment (try clicking on the "Fix me" link that appears for some issues), but from the issue list we could easily generate a few "easy issues" for people who want to contribute (with an eye towards GSoC and @dldotson's and @richardjgower's efforts to introduce more people at ASU to contributing to OS projects).
So, if you have a moment to spare, go to the URL above, pick something easy that you think ought to be fixed and raise an issue and label it with
- quantifiedcode (list of all open issues tagged "quantifiedcode")
- difficulty-easy
- help wanted
- ... and any other appropriate labels
As the issue title use the SAME TITLE as the QC issue (to avoid duplication).
As example see https://github.com/MDAnalysis/mdanalysis/issues/564 --- assume that someone new to MDAnalysis will read the issue so be explicit and use links to the QC issue.