diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-03-27 09:31:42 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-03-30 10:41:13 +0300 |
commit | d44116a50e52335d009a2b47ff63ca7a7ca9720a (patch) | |
tree | c2b2363b82e80e3cdc3037b49df25f23720ee099 | |
parent | e932dbba5a4e0052f398f951e875cf31aebc5392 (diff) |
repository: adjust syntax to ignore linting warnings
Since having migrated to golangci-lint in 713d1e48 (Use golangci-lint
for static code analysis, 2020-02-06), there are two linting warnings
in `raw_changes_test.go` that ought to be ignored but arent due to using
the old "lint:ignore" syntax instead of the new "nolint:golint" one.
Adjust to use golangci-linter's syntax to fix these warnings.
-rw-r--r-- | internal/service/repository/raw_changes_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/service/repository/raw_changes_test.go b/internal/service/repository/raw_changes_test.go index 6b3ad06b9..76f1e2233 100644 --- a/internal/service/repository/raw_changes_test.go +++ b/internal/service/repository/raw_changes_test.go @@ -371,13 +371,13 @@ func TestGetRawChangesInvalidUTF8Paths(t *testing.T) { for _, rawChange := range msg.GetRawChanges() { if string(rawChange.GetOldPathBytes()) == nonUTF8Filename { oldPathFound = true - //lint:ignore SA1019 gitlab.com/gitlab-org/gitaly/issues/1746 + //nolint:golint // gitlab.com/gitlab-org/gitaly/issues/1746 require.Equal(t, rawChange.GetOldPath(), InvalidUTF8PathPlaceholder) } if string(rawChange.GetNewPathBytes()) == nonUTF8Filename { newPathFound = true - //lint:ignore SA1019 gitlab.com/gitlab-org/gitaly/issues/1746 + //nolint:golint // gitlab.com/gitlab-org/gitaly/issues/1746 require.Equal(t, rawChange.GetNewPath(), InvalidUTF8PathPlaceholder) } } |