On 30.11.2017 11:14, Josef Reidinger wrote:
Well, advantage against plain rubocop is that it
1. integrate more services -
seehttps://docs.codeclimate.com/docs/list-of-engines
Yes, I looked at that, and what Ruby-relevant things I see beyond rubocop is:
- brakeman: checks RAILS apps for security vulnerabilities
We don't use RAILS, so this does not appear to be relevant for us.
- bundler-audit: Find security vulnerabilities in your Ruby dependencies.
I would hope that our security team keeps looking for those things in the
entire distribution. Other Ruby development teams outside of SUSE don't have
that luxury, so that might really be a plus for them.
- codeclimate-duplication: Tries to find duplicate code. OK, that might be
useful if it really works. But the question is if it works across package
boundaries, e.g. between one YaST package and others. I am doubtful about that.
2. visualize it
https://codeclimate.com/github/yast/yast-storage-ng
Is that it, or is there more?
3. have rating, so it is not plain pass/fail, but allow to see something like
worse, but still not above limit.
https://codeclimate.com/github/yast/yast-storage-ng
https://codeclimate.com/github/yast/yast-installation
https://codeclimate.com/github/yast/yast-network
Hm...
4. provides badge for project, so it shows external people some idea about code
quality
If it's badges we are after, I can offer you as many as you like - colored,
black and white, whatever.
But I am not sure how many users and customers care about that. As Arvin
mentioned, the number and severity of bugs and our response time for a first
reaction and until a fix is released will matter a lot more to them. But none
of that is something we make easily accessible to the public.
Just my 2 Cents.
Kind regards
--
Stefan Hundhammer <[email protected]>
YaST Developer
SUSE Linux GmbH
GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)
Maxfeldstr. 5, 90409 Nürnberg, Germany
--
To unsubscribe, e-mail: [email protected]
To contact the owner, e-mail: [email protected]
