Skip to content
Oliver Beckstein edited this page Dec 11, 2015 · 5 revisions

Code Issues

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.

quantifiedcode: MDAnalysis

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.

Easy issues for new contributors

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

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.

Clone this wiki locally