Skip to content

Fix for CI Lint failure#147

Merged
ritikrajdev merged 3 commits intomainfrom
fix/ci-failure
Mar 12, 2026
Merged

Fix for CI Lint failure#147
ritikrajdev merged 3 commits intomainfrom
fix/ci-failure

Conversation

@ritikrajdev
Copy link
Contributor

Description

This PR aims to provide fix for ci lint failure blocking the PR merges.

Related Issue

How Has This Been Tested?

@ritikrajdev ritikrajdev requested a review from a team as a code owner March 9, 2026 00:54
- name: Run golangci-lint
uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20
with:
version: v2.8.0

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anchoring the linter action to a specific action may make the updates to golangci/golangci-lint-action.
Were the failures happening due to 1.13 ? If so, we can probably look into increasing the go version itself

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go multierror recommends inbuilt errors library to be use in case of newer versions. I would not recommend upgrading to newer version here.

@ritikrajdev ritikrajdev merged commit 996c9fc into main Mar 12, 2026
17 checks passed
@ritikrajdev ritikrajdev deleted the fix/ci-failure branch March 12, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants