security-profiles-operator icon indicating copy to clipboard operation
security-profiles-operator copied to clipboard

Bump golangci-lint to v1.48.0

Open saschagrunert opened this issue 2 years ago • 4 comments

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

This incorporates new linters as well as the configuration update. New lint reports have been fixed as well.

Which issue(s) this PR fixes:

None

Does this PR have test?

None

Special notes for your reviewer:

None

Does this PR introduce a user-facing change?

None

saschagrunert avatar Aug 05 '22 07:08 saschagrunert

Codecov Report

Merging #1072 (761bbe7) into main (b33aada) will increase coverage by 0.03%. The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #1072      +/-   ##
==========================================
+ Coverage   50.43%   50.46%   +0.03%     
==========================================
  Files          42       42              
  Lines        4761     4756       -5     
==========================================
- Hits         2401     2400       -1     
+ Misses       2281     2278       -3     
+ Partials       79       78       -1     

codecov-commenter avatar Aug 05 '22 07:08 codecov-commenter

/retest

pjbgf avatar Aug 06 '22 10:08 pjbgf

Rebased and fixed a bunch of other lint reports.

saschagrunert avatar Aug 22 '22 07:08 saschagrunert

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JAORMX, jhrozek, pjbgf, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • ~~OWNERS~~ [JAORMX,jhrozek,pjbgf,saschagrunert]

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Aug 22 '22 09:08 k8s-ci-robot