Skip to content

Update Go testing#186

Open
SuperQ wants to merge 1 commit into
bradfitz:masterfrom
SuperQ:superq/go_bump
Open

Update Go testing#186
SuperQ wants to merge 1 commit into
bradfitz:masterfrom
SuperQ:superq/go_bump

Conversation

@SuperQ

@SuperQ SuperQ commented Apr 4, 2025

Copy link
Copy Markdown
  • Update CI to test all latest versions.
  • Bump GitHub actions and pin versions for supply chain security.
  • Enable dependabot to update GitHub actions.
  • Only build on push to master.

@SuperQ

SuperQ commented Apr 4, 2025

Copy link
Copy Markdown
Author

👓 @dormando

@dormando

dormando commented Apr 4, 2025

Copy link
Copy Markdown
Collaborator

I'm (personally) very conservative when bumping minimum versions for things. Can you add justifications for it in the commits/PR at least? Maybe separate it out if it's not necessary?

@SuperQ

SuperQ commented Apr 4, 2025

Copy link
Copy Markdown
Author

I've also tried to be more conservative about bumping minimum versions. But the Go upstream only supports the latest 2 releases.

So I typically recommend most libraries follow this policy.

But I'm happy to increase the support window here.

@dormando

dormando commented Apr 4, 2025

Copy link
Copy Markdown
Collaborator

Thanks! we can tighten that later once I better understand the userbase. memcached has a lot of users with fairly static environments (from bureaucracy or lack of time) but they sometimes need to urgently get a bug fixed.

@dormando dormando added this to the Minor / misc milestone Apr 4, 2025
* Update CI to test all latest versions.
* Bump GitHub actions and pin versions for supply chain security.
* Enable dependabot to update GitHub actions.
* Only build on push to master.
* Improve actions security settings.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ

SuperQ commented Jul 12, 2026

Copy link
Copy Markdown
Author

I updated to the latest actions and added a couple additional security changes.

I also added a branch filter on push to avoid double-runs on pull requests that are local (non-fork) branch pushes.

@SuperQ

SuperQ commented Jul 12, 2026

Copy link
Copy Markdown
Author

Maybe @bradfitz can review this?

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.

3 participants