Skip to content
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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fbm3307
Copy link
Contributor

@fbm3307 fbm3307 commented Dec 30, 2024

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

…rget

Signed-off-by: Feny Mehta <fbm3307@gmail.com>
Copy link

codecov bot commented Dec 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.23%. Comparing base (34f3205) to head (fb6bff1).

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>
Copy link
Contributor

@mfrancisc mfrancisc left a 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
Copy link
Contributor

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 ?

Copy link
Contributor Author

@fbm3307 fbm3307 Dec 31, 2024

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

Copy link
Contributor

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>
Copy link

sonarqubecloud bot commented Jan 7, 2025

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