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

PWX-33069_23.7.0: upgrading golang and tools (#829) (#1215) (#1227) #1227

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

zoxpx
Copy link
Collaborator

@zoxpx zoxpx commented Aug 24, 2023

Fixing Makefiles to use golangci-lint as standalone tool
Upgrading golang to 1.20.7

Signed-off-by: Zoran Rajic zox@portworx.com

What this PR does / why we need it:

This is a cherry-pick of #1215 into px-rel-23.7.0 branch

Which issue(s) this PR fixes (optional)
Closes # PWX-33069

Special notes for your reviewer:

* fixing makefile rules for linter-tools  (golangci-lint, errcheck, staticcheck)
* removed unused gomock tool

Signed-off-by: Zoran Rajic <zrajic@purestorage.com>
* also updating linter-tools and integration-tests

Signed-off-by: Zoran Rajic <zox@portworx.com>

Manually fixed Conflicts:
	Makefile
	go.mod
	go.sum
	vendor/github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring/v1/prometheus_types.go
	vendor/modules.txt
@zoxpx zoxpx self-assigned this Aug 24, 2023
@zoxpx zoxpx changed the title PWX-33069_23.7.0: upgrading golang and tools PWX-33069_23.7.0: upgrading golang and tools (#829) (#1215) (#1227) Aug 24, 2023
@zoxpx
Copy link
Collaborator Author

zoxpx commented Aug 24, 2023

These commits are cherry-picked from master that have already been reviewed -- merging the cherry-picks

@zoxpx zoxpx merged commit 619b99b into px-rel-23.7.0 Aug 24, 2023
5 checks passed
@zoxpx zoxpx deleted the px-23.7.0_upgrading_golang_and_tools branch August 24, 2023 06:49
@zoxpx
Copy link
Collaborator Author

zoxpx commented Aug 24, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant