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

gqlgen jamietanna rocks edition (x/tools temporary update) #3528

Closed
wants to merge 2 commits into from

Conversation

StevenACoffman
Copy link
Collaborator

@StevenACoffman StevenACoffman commented Feb 10, 2025

For @jamietanna (who is awesome) based on #3505

We cannot upgrade gqlgen to a more recent x/tools until GCP AppEngine updates to support Go 1.23 (ugh... ) but maybe having this random PR with this commit trapped in amber for all time will allow you to try out Go 1.24 rc2 using a replace directive in your project:

go mod edit -replace github.com/99designs/gqlgen=github.com/99designs/gqlgen@a3dcd48596adbbba527c0d01bbcf78056e25783e
go mod tidy
  • chore(deps): bump golang.org/x/tools from 0.24.0 to 0.29.0
  • Go mod tidy for examples

You can see the issue in his project here:
https://gitlab.com/tanna.dev/dependency-management-data/-/issues/643

dependabot bot and others added 2 commits February 10, 2025 11:50
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.24.0 to 0.29.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](golang/tools@v0.24.0...v0.29.0)

---
updated-dependencies:
- dependency-name: golang.org/x/tools
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Steve Coffman <[email protected]>
@StevenACoffman StevenACoffman changed the title jamietanna rocks gqlgen jamietanna rocks edition (x/tools temporary update) Feb 10, 2025
@coveralls
Copy link

Coverage Status

coverage: 73.389% (+0.008%) from 73.381%
when pulling a3dcd48 on jamietanna_rocks
into 325c04a on master.

@StevenACoffman
Copy link
Collaborator Author

Hey, I'm going to leave the branch for a while, but I think this PR is no longer necessary.

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.

2 participants