search

ansible / mazer

Experimental Ansible Galaxy Content Manager
GNU General Public License v3.0
114 stars 18 forks source link

misc cleanup #89

Closed alikins closed 6 years ago

alikins commented 6 years ago
SUMMARY

Misc cleanups, largely based on https://lgtm.com/projects/g/ansible/mazer/alerts/?mode=list and https://app.codacy.com/app/alikins/mazer/issues?step=0

ISSUE TYPE
GALAXY CLI VERSION
Ansible Galaxy CLI, version0.1.0
Linux, newswoop, 4.16.6-202.fc27.x86_64, #1 SMP Wed May 2 00:09:32 UTC 2018, x86_64
3.6.5 (default, Apr  4 2018, 15:01:18) 
[GCC 7.3.1 20180303 (Red Hat 7.3.1-5)]/home/adrian/venvs/galaxy-cli-py3/bin/python
Using /home/adrian/.ansible/mazer.yml as config file
ADDITIONAL INFORMATION
s0 commented 6 years ago

Hi @alikins

Glad to see that you're addressing alerts that have been found by lgtm.com. Did you know that we have pull request support that will check whether PRs would introduce new alerts, to help you prevent them in the first place. You can enable it for mazer here: https://lgtm.com/projects/g/ansible/mazer/ci/

We also have badges (that you can see from the link above) that you can embed into your README if you like, such as: Total Alerts

Let me know if you have any questions.