HERA-Team / hera_mc

The HERA monitor and control (M&C) system.

BSD 2-Clause "Simplified" License

3 stars 3 forks source link

Remove linting job from workflow #593

Closed plaplant closed 2 years ago

plaplant commented 2 years ago

Description

Now that we have a hook to pre-commit.ci as part of our CI, we don't need to run a separate linting job. This PR removes it from the series of GitHub actions.

Motivation and Context

Previously we were running our linting jobs as a GitHub action. We don't need to do this anymore.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Schema change (any change to the SQL tables)
[ ] New feature without schema change (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Version change
[x] Build or continuous integration change
[ ] Other

Checklist:

Build or continuous integration change checklist:

[ ] If required or optional dependencies have changed (including version numbers), I have updated the readme to reflect this.

codecov[bot] commented 2 years ago

Codecov Report

Merging #593 (28a728b) into main (ff2bf62) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #593   +/-   ##
=======================================
  Coverage   97.76%   97.76%           
=======================================
  Files          35       35           
  Lines        5110     5110           
=======================================
  Hits         4996     4996           
  Misses        114      114

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ff2bf62...28a728b. Read the comment docs.