From eb36a4554ee9a9d15954f08dd37deee54385e29c Mon Sep 17 00:00:00 2001 From: TheFox0x7 Date: Fri, 27 Jun 2025 07:59:55 +0200 Subject: enforce nolint scope (#34851) enable nolintlint scope requirement add comments to new directives so it's more obvious why they are in place --- I can also toggle the mandatory comments on if that's something of interest. --------- Co-authored-by: wxiaoguang Co-authored-by: Giteabot --- modules/commitstatus/commit_status.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'modules/commitstatus/commit_status.go') diff --git a/modules/commitstatus/commit_status.go b/modules/commitstatus/commit_status.go index 12004474ed..a0ab4e7186 100644 --- a/modules/commitstatus/commit_status.go +++ b/modules/commitstatus/commit_status.go @@ -5,7 +5,7 @@ package commitstatus // CommitStatusState holds the state of a CommitStatus // swagger:enum CommitStatusState -type CommitStatusState string //nolint +type CommitStatusState string //nolint:revive // export stutter const ( // CommitStatusPending is for when the CommitStatus is Pending @@ -56,7 +56,7 @@ func (css CommitStatusState) IsSkipped() bool { return css == CommitStatusSkipped } -type CommitStatusStates []CommitStatusState //nolint +type CommitStatusStates []CommitStatusState //nolint:revive // export stutter // According to https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28#get-the-combined-status-for-a-specific-reference // > Additionally, a combined state is returned. The state is one of: -- cgit v1.2.3