Skip to content

Conversation

@utarwyn
Copy link
Contributor

@utarwyn utarwyn commented Sep 19, 2023

This PR upgrades some files and dependencies based on what are done in other Drone plugins.

  • Bump Go from 1.19 to 1.21
  • Upgrade dependencies
  • Update gitignore
  • Remove custom golangci-lint configuration

Should be integrated before #155 in order to fix the build.

@donny-dont
Copy link
Contributor

Why is the .golangci.yml deleted? Also why Go 1.20 instead of Go 1.21?

@utarwyn
Copy link
Contributor Author

utarwyn commented Sep 19, 2023

@donny-dont
Why is the .golangci.yml deleted? Also why Go 1.20 instead of Go 1.21?

This file seems to be deleted in other maintained plugins, no? Without the file, golangci-lint works with the default configuration and linters (see more here). Let me know if you still want to keep it (but we will need to update it).

Yes I can upgrade to Go 1.21 directly if you want

@utarwyn
Copy link
Contributor Author

utarwyn commented Sep 27, 2023

Hello @donny-dont @tphoney,
I have updated the plugin to Go 1.21. Can you review the PR?

@tphoney tphoney merged commit ab04320 into drone-plugins:master Sep 28, 2023
@utarwyn utarwyn deleted the plugin-upgrade branch September 28, 2023 10:49
@tphoney
Copy link

tphoney commented Sep 28, 2023

thanks for the PR @utarwyn merged.

@utarwyn
Copy link
Contributor Author

utarwyn commented Sep 28, 2023

Thank you for the quick review! 🚀

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.

3 participants