Uploaded image for project: 'Apache Fineract'
  1. Apache Fineract
  2. FINERACT-1246

New :integration-tests module should have Checkstyle enabled

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None

    Description

      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

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              vorburger Michael Vorburger
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: