Skip to content

DO NOT MERGE: probe the MySQL job on Rails 8.1 - #229

Closed
Fivell wants to merge 2 commits into
masterfrom
ci/try-mysql-rails-8-1
Closed

Fivell wants to merge 2 commits into
masterfrom
ci/try-mysql-rails-8-1

Conversation

@Fivell

@Fivell Fivell commented Oct 1, 2026

Copy link
Copy Markdown
Member

Throwaway branch. The only purpose is to let the GitHub runner's mysql:8.0 service answer a question that could not be answered locally: does the MySQL-backed job pass on Rails 8.1?

Context: PR #228 moved the PostgreSQL jobs to Rails 8.1 but deliberately left test-mysql on 8.0, because no MySQL server was available locally and pushing an unverified change was the wrong trade. This probe closes that gap. If the MySQL job is green here, the change moves onto #228 and this PR is closed without merging.

Rails 8.0 goes EOL 2026-11-07, which is why the job should not stay there.

Fivell added 2 commits October 1, 2026 13:49
Rails 7.2 reached end of life on 2026-08-09 (endoflife.date/rails), so the
SQLite matrix now tests Rails 8.0 and 8.1 only. The ActiveAdmin 4 include
leg that pinned Rails 7.2.0 goes with it.

Both `exclude:` rules are also removed. Their stated reason -- "ActiveAdmin
3.2.0 predates Rails 8 support" -- is stale: the matrix pins `~> 3.2.0`,
which resolves to activeadmin 3.2.5, and that installs and passes the suite
on both Rails 8.0.5.1 and 8.1.4 (66 examples, 0 failures each). The rules
were hiding working coverage.

The two PostgreSQL jobs move from Rails 8.0.0 to 8.1.0, both verified
locally against a real PostgreSQL. The MySQL job stays on 8.0.0, which is
supported until 2026-11-07, so one DB-backed job still covers the oldest
supported Rails.
@Fivell

Fivell commented Oct 1, 2026

Copy link
Copy Markdown
Member Author

Probe done, closing without merging. The MySQL job passes on Rails 8.1: 66 examples, 0 failures, with mysql2 0.5.7 built natively and the dummy app configured for MySQL via ENV['DB'] (job log). The 67th example in the PostgreSQL runs is the ENV['DB'] == 'postgres' native-upsert spec, so the counts line up.

The change is now on #228, so test-mysql no longer sits on Rails 8.0 (EOL 2026-11-07).

@Fivell Fivell closed this Oct 1, 2026
@Fivell
Fivell deleted the ci/try-mysql-rails-8-1 branch October 1, 2026 12:14
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