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

[chore] bump to Go1.23 #3570

Closed

Conversation

NyaaaWhatsUpDoc
Copy link
Member

this is a copy of #3258 but rebased on current main and updated to latest golangci-lint version, and with docker image version strings updated to just use the latest minor version of a particular major image version (this ensures we stay up-to-date with bugfixes but without breaking changes).

i made this a separate PR just because i wasn't totally comfortable doing a force push to something that wasn't my PR :p

@@ -5,7 +5,7 @@
set -eu

swagger_cmd() {
go run github.com/go-swagger/go-swagger/cmd/swagger "$@"
go run ./vendor/github.com/go-swagger/go-swagger/cmd/swagger "$@"
Copy link
Member Author

Choose a reason for hiding this comment

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

always tests swagger using the vendored version

project_name: gotosocial
version: 2

# https://goreleaser.com/customization/hooks/
before:
hooks:
# generate the swagger.yaml file using go-swagger and bundle it into the assets directory
- swagger generate spec --scan-models --exclude-deps -o web/assets/swagger.yaml
- go run ./vendor/github.com/go-swagger/go-swagger/cmd/swagger generate spec --scan-models --exclude-deps -o web/assets/swagger.yaml
Copy link
Member Author

Choose a reason for hiding this comment

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

always generates swagger using the vendored version

@NyaaaWhatsUpDoc
Copy link
Member Author

moved back to the original PR #3258 now

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