Skip to content

Update docs to use go 1.24 tool dependencies instead of internal/tools.go pattern#3741

Merged
StevenACoffman merged 2 commits into
99designs:masterfrom
deitrix:tool-dependency-docs
Nov 18, 2025
Merged

Update docs to use go 1.24 tool dependencies instead of internal/tools.go pattern#3741
StevenACoffman merged 2 commits into
99designs:masterfrom
deitrix:tool-dependency-docs

Conversation

@deitrix

@deitrix deitrix commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

Resolves #3740

This PR aims to update the gqlgen documentation to use the new go tool dependency functionality introduced in Go 1.24, as opposed to the older internal/tools.go pattern.

Now, the recommended way to add tool dependencies is:

go get -tool github.com/99designs/gqlgen

These tools can then be ran directly using go tool. So, any occurrences of:

go run github.com/99designs/gqlgen

Have now been replaced with:

go tool gqlgen

Hopefully I caught them all!

@deitrix deitrix force-pushed the tool-dependency-docs branch from b6ec3b3 to ec7cac4 Compare June 17, 2025 19:04
@deitrix deitrix marked this pull request as ready for review June 17, 2025 19:14
@StevenACoffman StevenACoffman marked this pull request as draft June 18, 2025 01:52
@StevenACoffman

StevenACoffman commented Jun 18, 2025

Copy link
Copy Markdown
Collaborator

Thanks! I marked it as draft just to mark it as "don't merge" until Go 1.25 is released, if that's ok?

@coveralls

coveralls commented Jun 18, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 0.0%. remained the same
when pulling 1535c94 on deitrix:tool-dependency-docs
into b7a335f on 99designs:master.

@deitrix

deitrix commented Jun 18, 2025

Copy link
Copy Markdown
Contributor Author

Thanks! I marked it as draft just to mark it as "don't merge" until Go 1.25 is released, if that's ok?

Absolutely, no problem! 😄

@StevenACoffman StevenACoffman marked this pull request as ready for review November 17, 2025 16:47
@StevenACoffman StevenACoffman self-requested a review as a code owner November 17, 2025 16:47
@StevenACoffman

StevenACoffman commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

@deitrix Do you mind freshening this up a bit?

@deitrix deitrix force-pushed the tool-dependency-docs branch 2 times, most recently from ec7cac4 to cd4bdae Compare November 17, 2025 18:05
@deitrix deitrix force-pushed the tool-dependency-docs branch from cd4bdae to d3e98e6 Compare November 17, 2025 18:10
@deitrix

deitrix commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

@StevenACoffman I've rebased the branch and given it a once over. All still looks good to me!

@StevenACoffman StevenACoffman merged commit c442432 into 99designs:master Nov 18, 2025
19 checks passed
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.

Proposal: Getting started docs - Use new Go 1.24 tool dependencies for CLI

3 participants