marquez icon indicating copy to clipboard operation
marquez copied to clipboard

[DOCS] Add linter to check for headers in Java files

Open merobi-hub opened this issue 2 years ago • 1 comments

Signed-off-by: Michael Robinson [email protected]

Problem

A linter as part of the Github Actions workflows could be used to automate the header checking process after the copyright/license PR is merged.

Note: this should be merged only after #1996 has been merged.

Solution

This PR adds the checkstyles linter to .github/workflows for the purpose of checking Java files for a header.

Note: All database schema changes require discussion. Please link the issue for context.

Checklist

  • [x] You've signed-off your work
  • [ ] Your changes are accompanied by tests (if relevant)
  • [x] Your change contains a small diff and is self-contained
  • [ ] You've updated any relevant documentation (if relevant)
  • [ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
  • [ ] You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)

merobi-hub avatar Jun 23 '22 14:06 merobi-hub

Codecov Report

Merging #2021 (c3b3cdf) into main (7a7a2a8) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2021   +/-   ##
=========================================
  Coverage     78.89%   78.89%           
  Complexity     1014     1014           
=========================================
  Files           197      197           
  Lines          5548     5548           
  Branches        421      421           
=========================================
  Hits           4377     4377           
  Misses          724      724           
  Partials        447      447           

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

codecov[bot] avatar Jun 23 '22 14:06 codecov[bot]