ci: add dynamic perl version matrix testing#1786
Draft
toddr-bot wants to merge 1 commit intoPerlDancer:mainfrom
Draft
ci: add dynamic perl version matrix testing#1786toddr-bot wants to merge 1 commit intoPerlDancer:mainfrom
toddr-bot wants to merge 1 commit intoPerlDancer:mainfrom
Conversation
…esting Add two new CI jobs inspired by YAML-Syck's testsuite.yml: - perl-versions: uses perl-actions/perl-versions@v1 to dynamically discover all available perl versions since 5.14 (including devel) - test-perl-matrix: installs the built Dancer2 artifact across every discovered perl version using perldocker/perl-tester containers This enhances (does not replace) the existing CI jobs, providing visibility into perl version compatibility across the full spectrum. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Add dynamic perl version matrix to CI for comprehensive compatibility testing across all available perl versions.
Why
The existing CI tests only 3 perl versions (latest, 5.36, 5.20) with many versions commented out in
test-on-clean-perl. This leaves gaps in compatibility visibility — breakage on perl 5.22, 5.26, 5.30, etc. would go unnoticed. Inspired by YAML-Syck's approach, the new jobs dynamically discover all available perl versions so the matrix stays current without manual updates.How
Two new jobs added (existing jobs untouched):
perl-versions: Usesperl-actions/perl-versions@v1to discover all perl versions since 5.14 (including devel releases), outputting them as a JSON array.test-perl-matrix: Consumes that list as a dynamic matrix, installing the Dancer2 artifact (frombuild-in-container) on each version viaperldocker/perl-testercontainers.The
since-perl: 5.14floor was chosen because Dancer2 declaresuse 5.12.0and 5.14 is the oldest version with reliable Docker images.with-devel: truegives visibility into upcoming perl releases.Testing
🤖 Generated with Claude Code
Quality Report
Changes: 1 file changed, 42 insertions(+)
Code scan: clean
Tests: skipped
Branch hygiene: clean
Generated by Kōan post-mission quality pipeline