Michael Vorburger created FINERACT-1246:
-------------------------------------------

             Summary: New :integration-tests module should have Checkstyle 
enabled
                 Key: FINERACT-1246
                 URL: https://issues.apache.org/jira/browse/FINERACT-1246
             Project: Apache Fineract
          Issue Type: Improvement
            Reporter: Michael Vorburger


FINERACT-1209 introduce a new {{:integration-tests}} module (big refactoring).

In https://github.com/apache/fineract/pull/1465/files [~aleks] had to add a new 
suppression (exclusion) for {{/integration-tests/}} to 
{{config/checkstyle/suppressions.xml}}. (Apparently the integration tests were 
not yet covered by Checkstyle even in the old location?)

The goal of this issue is to remove that suppression and enable Checkstyle on 
ITs.

An intermediate solution could be to not fully enable it like on main code, but 
not entirely disable it either, but just start by enabling the CS rules which 
already pass for ITs, maybe fix a few trivial ones, but leave those with many 
violations for separate future fixing.

[~manthan] FYI



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to