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

Make various improvements to our GitHub workflow #347

Merged
merged 7 commits into from
Sep 4, 2024

Conversation

NullHypothesis
Copy link
Contributor

@NullHypothesis NullHypothesis commented Sep 3, 2024

In particular:

  • Test Go 1.23.
  • Use the latest version for our Ubuntu and macOS runners.
  • Remove the cache action as it's no longer needed.
  • A few fixes for the test to run on new macOS instances.

I recommend reviewing the PR commit by commit.

There are no significant downsides to using the latest version for both
Ubuntu and macOS runners.
Go 1.23 was released last month.  Time to test it!
These latest versions may (or may not) benefit us and they certainly
shouldn't harm us.
Ever since v4, `actions/setup-go` can cache dependencies as long as
`actions/checkout` is run first, which we do:
https://github.com/actions/setup-go#v4
@NullHypothesis NullHypothesis force-pushed the phw/improve-github-workflow branch 2 times, most recently from 6f2ba78 to 1a070d8 Compare September 3, 2024 17:33
@NullHypothesis NullHypothesis marked this pull request as ready for review September 3, 2024 22:34
@NullHypothesis NullHypothesis merged commit 6cb6409 into master Sep 4, 2024
10 checks passed
@NullHypothesis NullHypothesis deleted the phw/improve-github-workflow branch September 4, 2024 14:34
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