29 Nov
2017
29 Nov
'17
15:45
On Wednesday 2017-11-29 16:24, Ancor Gonzalez Sosa wrote:
The point is that there is another new check that is yelling to us and, since there is no 1:1 Rubocop equivalent we need to take a decision. Codeclimate thinks that each class should not have more than 20 methods. With my pull request, Y2Storage::Md has grown to 23 methods.
What should we do? Try to honor this new check (which means rethinking some of our classes, MD is simply today's example)? Disable it completely? Raise the threshold to any value you guys find reasonable?
I would not spend too much time with this. Just disable it. Steffen -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org