Uploaded image for project: 'Daffodil'
  1. Daffodil
  2. DAFFODIL-2799

Separate "lint" checks from "OS/JDK" matrix checks

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.5.0
    • 3.5.0
    • Infrastructure
    • None

    Description

      Contributors must feel discouraged to make a PR change and see 3 out of 6 matrix jobs fail across the board.  Let's move the Rat, OSGi, scalafmt, and sonarcloud checks which aren't OS-specific or JDK-specific into their own lint job inside main.yml separate from the OS/JDK checks.  Separating the lint checks would also let the OS/JDK matrix jobs finish 1-2 minutes quicker.  In order to give contributors and reviewers as much information as possible about a PR, let's configure the new job to always run all the lint checks even if one lint check fails and always run all the OS/JDK checks even if any of the lint checks fail since we still want to know whether all the OS/JDK tests pass as well as whether all the lint checks pass.

      Attachments

        Activity

          People

            Unassigned Unassigned
            jinterrante John Interrante
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: