Skip to content

Refresh action pins; add MySQL examples - #11

Merged
bashhack merged 2 commits into
mainfrom
tidy
Oct 22, 2026
Merged

bashhack merged 2 commits into
mainfrom
tidy

Conversation

@bashhack

Copy link
Copy Markdown
Owner

Two small things.

  • The checkout, setup-go, cache and upload-artifact actions move from their May 2025 commits to the current releases (v7.0.1, v7.0.0, v6.1.0, v7.0.2), pinned by SHA as before. golangci-lint-action was already current.
  • mysql/example_test.go shows the MySQL API on pkg.go.dev: Setup, a custom initializer, a template in TestMain, keep-on-failure. They are compile-checked rather than run: a runnable example cannot skip when no MySQL server is reachable, and the module's MySQL tests skip rather than fail on a Postgres-only machine, so running examples would break that.

checkout, setup-go, cache and upload-artifact move from their May 2025
commits to the current releases (v7.0.1, v7.0.0, v6.1.0, v7.0.2).

mysql/example_test.go shows the MySQL API: Setup, a custom initializer,
a template in TestMain, keep-on-failure. The examples are compile-checked
rather than run, since a runnable example cannot skip when no MySQL
server is reachable, and this module's MySQL tests skip rather than fail
on a machine that only runs PostgreSQL.
The pool records a prepared database before cloning into it, so two
records can exist while the second CREATE DATABASE has not run yet; CI
killed the holder at that instant and found one database. The test now
waits for both databases to exist, which also makes a kill mid-clone a
case the sweep is seen to handle.
@bashhack
bashhack merged commit 7964c18 into main Oct 22, 2026
1 check passed
@bashhack
bashhack deleted the tidy branch October 22, 2026 06:52
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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