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

Update checksums in go.work.sum + tidy everything #543

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

brandur
Copy link
Contributor

@brandur brandur commented Aug 21, 2024

So this is going to be a small annoyance with workspaces. After cutting
a new release, Go really wants to put the new checksums in
go.work.sum, and even having the LSP running will update it for you
automatically. If you revert the changes, the LSP will update them
again, which will block you from rebasing and switching branches, which
as you can imagine, is great fun.

So here, add the new checksums to go.work.sum, something which we'll
probably have to do after each release goes out.

Also, since we're doing a checksum change anyway, I also ran make tidy
to clean up all the package go.sum files.

So this is going to be a small annoyance with workspaces. After cutting
a new release, Go **really** wants to put the new checksums in
`go.work.sum`, and even having the LSP running will update it for you
automatically. If you revert the changes, the LSP will update them
again, which will block you from rebasing and switching branches, which
as you can imagine, is great fun.

So here, add the new checksums to `go.work.sum`, something which we'll
probably have to do after each release goes out.

Also, since we're doing a checksum change anyway, I also ran `make tidy`
to clean up all the package `go.sum` files.
@brandur
Copy link
Contributor Author

brandur commented Aug 21, 2024

ty.

@brandur brandur merged commit 6a0d904 into master Aug 21, 2024
14 checks passed
@brandur brandur deleted the brandur-update-go-work-sum branch August 21, 2024 02:33
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