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

Migrate to mise for local/CI and bump versions #111

Merged
merged 1 commit into from
Jan 3, 2025
Merged

Migrate to mise for local/CI and bump versions #111

merged 1 commit into from
Jan 3, 2025

Conversation

rliebz
Copy link
Owner

@rliebz rliebz commented Jan 3, 2025

Normally I wouldn't want to introduce an extra dependency in CI, but I think this would be extremely easy to undo if it ever started to cause problems. The main reason to use it in actions is to avoid declaring tool versions in two places that need to be kept in sync.

The major benefit here is that for tools like golangci-lint and goreleaser, we now use an official release binary rather than building from source, which was never officially supported or recommended. Not that it's caused problems yet, but better to play it safe.

This also has the benefit of ensuring that the version of golangci-lint that runs in my editor is the same as the version the project wants me to use.

Normally I wouldn't want to introduce an extra dependency in CI, but I
think this would be extremely easy to undo if it ever started to cause
problems. The main reason to use it in actions is to avoid declaring
tool versions in two places that need to be kept in sync.

The major benefit here is that for tools like golangci-lint and
goreleaser, we now use an official release binary rather than building
from source, which was never officially supported or recommended. Not
that it's caused problems yet, but better to play it safe.

This also has the benefit of ensuring that the version of golangci-lint
that runs in my editor is the same as the version the project wants me
to use.
@rliebz rliebz merged commit 7208eb2 into main Jan 3, 2025
5 checks passed
@rliebz rliebz deleted the mise branch January 3, 2025 17:46
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.

1 participant