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

To fix goreleaser #2062

Merged
merged 1 commit into from
Apr 6, 2024
Merged

Conversation

cniackz
Copy link
Contributor

@cniackz cniackz commented Apr 5, 2024

Objective:

To fix goreleaser test, that's it!

Additionally use proper flag:

  • DEPRECATED: --skip-publish was deprecated in favor of --skip=publish, check https://goreleaser.com/deprecations#-skip for more details

Other files:

Since plugin is not deprecated, these files got updated as well:

  • kubectl-minio/go.mod
  • kubectl-minio/go.sum

@cniackz cniackz self-assigned this Apr 5, 2024
@cniackz cniackz force-pushed the fix-goreleaser-apr5 branch 2 times, most recently from 0af9753 to 9be330e Compare April 5, 2024 21:25
@cniackz cniackz changed the title Fix goreleaser To fix goreleaser Apr 5, 2024
@cniackz cniackz added the dependencies Pull requests that update a dependency file label Apr 5, 2024
@cniackz cniackz requested a review from reivaj05 April 5, 2024 22:07
Copy link
Contributor

@allanrogerr allanrogerr left a comment

Choose a reason for hiding this comment

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

This seems to be accomplishing what #2051 is already doing.
Could you check with @dvaldivia to add you goreleaser change?

@allanrogerr
Copy link
Contributor

This does not seem necessary @cniackz - see goreleaser run here #2051

@cniackz
Copy link
Contributor Author

cniackz commented Apr 6, 2024

@allanrogerr, this is unrelated to the 2051 PR. This PR is simply to fix the test, not to deprecate anything. Initially, I thought the plugin was deprecated, but now I see that the PR is still open. We still need this change to fix our CI/CD pipeline. Please review it again.

@cniackz
Copy link
Contributor Author

cniackz commented Apr 6, 2024

Thank you anyway for the review @allanrogerr because it made me realize we never deprecated the plugin 👍

@harshavardhana harshavardhana merged commit 52be236 into minio:master Apr 6, 2024
26 checks passed
@cniackz cniackz deleted the fix-goreleaser-apr5 branch April 8, 2024 14:13
@cniackz
Copy link
Contributor Author

cniackz commented Apr 8, 2024

Thank you guys ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants