-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KUBESAW-173: Create GH Actions workflow running verify-replace-run target #441
base: master
Are you sure you want to change the base?
Conversation
…rget Signed-off-by: Feny Mehta <fbm3307@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #441 +/- ##
=======================================
Coverage 78.23% 78.23%
=======================================
Files 49 49
Lines 2444 2444
=======================================
Hits 1912 1912
Misses 478 478
Partials 54 54 |
Signed-off-by: Feny Mehta <fbm3307@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice Job 👍
I have only one question.
go-version-file: go.mod | ||
|
||
- name: Install golangci-lint | ||
run: go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.52.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as discussed - my "concern" is that some projects are using golanci-lint v1.56.2 ( see reg-svc , member operator ) . Should we first upgrade all projects to 1.56 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did try using v1.56.2 here, but it failed with error that it required go 1.21 , so i am a bit confused now, as to how go-lint v1.56.2 works in reg-svc, member-operator.
it may be probably because of pkg/slices as i see that in error, but unsure about this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe @xcoulon knows better
Signed-off-by: Feny Mehta <fbm3307@gmail.com>
Signed-off-by: Feny Mehta <fbm3307@gmail.com>
Signed-off-by: Feny Mehta <fbm3307@gmail.com>
Quality Gate passedIssues Measures |
This PR creates GH Actions running verify-replace-run target in api and toolchain-common repositories, making sure that the change introduced in the PR doesn't break other repos that depend on that.
Similar PR