Skip to content

test: prepare release-0-9 CI for routing compatibility - #1498

Merged
bf4 merged 2 commits into
release-0-9from
fix_09_ci
Oct 22, 2026
Merged

bf4 merged 2 commits into
release-0-9from
fix_09_ci

Conversation

@bf4

@bf4 bf4 commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

Extracted from #1497.

This PR contains the two test and CI fixes from that PR, without its Rails 8.1 routing implementation change:

  • Load Ruby's logger before Rails test dependencies so the Rails 6.0 and 6.1 matrix jobs can boot with the currently resolved gems.
  • Add the routing regression tests with path assertions based on the configured route formatter. The formatter fix in fix: backport Rails 8.1 routing keywords to release-0-9 #1497 edits a test file introduced there, so the complete corrected test file is included here.

Validation: rake test TEST=test/unit/routing_compatibility_test.rb passes locally (2 runs, 0 failures), and git diff --check is clean.

@bf4
bf4 merged commit 0a9c448 into release-0-9 Oct 22, 2026
8 checks passed
@bf4
bf4 deleted the fix_09_ci branch October 22, 2026 00:47
@bf4

bf4 commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator Author

xref #1496

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