search

canonical / hotsos

Software analysis toolkit. Define checks in high-level language and leverage library to perform analysis of common Cloud applications.
Apache License 2.0
31 stars 37 forks source link

pylintrc: re-enable too-many-lines #939

Closed brianphaley closed 1 month ago

brianphaley commented 1 month ago

To get under the 1000 line limit I did the following:

1) Moved exception classes to more specific files 2) Moved common definitions to common files 3) Moved ceph classes into more specific files 4) Moved unit tests to more specific files, following the model of having the unit test live in the same directory structure of the file it is testing 5) Moved some definitions into tests.utils 6) More, but all along these lines

There could probably be more breakdown of the tests into specific files, but I only did what was necessary to pass.

brianphaley commented 1 month ago

This is going to need re-base once some of these other patches merge, just wanted to get something out before the weekend

brianphaley commented 1 month ago

I split this change up into many patches, can abandon this now.