Skip to content

Conversation

@brandur
Copy link
Collaborator

@brandur brandur commented Mar 9, 2025

Here, move the API framework, which is currently duplicated in two
projects, over to its own repository so that we can reuse it between
them. It's new home will be riverapiframe [1].

[1] riverqueue/apiframe#1

@brandur brandur force-pushed the brandur-riverapiframe branch from 47af12b to 69b5424 Compare March 9, 2025 18:06
module riverqueue.com/riverui

go 1.22.0
go 1.23.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ugh, I tried not to change this, but the new repo really wanted to be Go 1.23 (and run of go mod tidy would change it to such), and when I brought that in here, of course it'd take this project to 1.23 as well.

Probably okay though since we don't really need to make as much an effort to support very old Go versions for this project, and currently only 1.23 and 1.24 are supported.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah I think it's fine, latest 2 Go versions is a reasonable and common stance.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yep, +1. SG.

@brandur brandur requested a review from bgentry March 9, 2025 18:09
module riverqueue.com/riverui

go 1.22.0
go 1.23.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah I think it's fine, latest 2 Go versions is a reasonable and common stance.

runs-on: ubuntu-latest
env:
GOLANGCI_LINT_VERSION: v1.63.4
GOLANGCI_LINT_VERSION: v1.64.6
Copy link
Contributor

Choose a reason for hiding this comment

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

We were on 1.64.4 elsewhere so we probably need to bump everywhere.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Luckily I don't think the new versions make that big of a difference (at least between 1.63 and 1.64). The main thing for some of them recently was getting in Go 1.24 support. But yeah, we should start rolling all the versions forward as we get to them.

Here, move the API framework, which is currently duplicated in two
projects, over to its own repository so that we can reuse it between
them. It's new home will be `riverapiframe` [1].

[1] riverqueue/apiframe#1
@brandur brandur force-pushed the brandur-riverapiframe branch from 69b5424 to 460a28f Compare March 10, 2025 05:41
@brandur
Copy link
Collaborator Author

brandur commented Mar 10, 2025

Thanks!

@brandur brandur merged commit c1d45ff into master Mar 10, 2025
13 checks passed
@brandur brandur deleted the brandur-riverapiframe branch March 10, 2025 05:51
@bgentry bgentry mentioned this pull request Mar 18, 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.

3 participants