Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
3.5.0
-
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.