Skip to content

ci: test Perl 5.10+ to match declared minimum#17

Draft
Koan-Bot wants to merge 2 commits into
masterfrom
koan.atoomic/ci-test-perl-5.10
Draft

ci: test Perl 5.10+ to match declared minimum#17
Koan-Bot wants to merge 2 commits into
masterfrom
koan.atoomic/ci-test-perl-5.10

Conversation

@Koan-Bot

Copy link
Copy Markdown
Collaborator

What

Lowers the CI perl-versions matrix from v5.14 to v5.10.

Why

The module declares MIN_PERL_VERSION = 5.010 but CI only tested 5.14+, leaving two major Perl releases (5.10, 5.12) completely uncovered. If a 5.14-only feature slipped in, no CI job would catch it — only CPAN smoke testers on old perls.

Addresses #16.

How

One-line change in .github/workflows/testsuite.yml: since-perl: v5.14v5.10.

The code only uses the // operator (5.10+), and [Test::MinimumVersion] in dist.ini already enforces max_target_perl = 5.010 at release time. This change adds runtime verification via actual test execution on 5.10 and 5.12.

Testing

  • Local test suite passes (48/48)
  • CI will validate the new matrix on this PR itself

🤖 Generated with Claude Code

The module declares MIN_PERL_VERSION = 5.010 in dist.ini and
Makefile.PL, but CI started at 5.14 — leaving 5.10 and 5.12
completely untested. Lower since-perl to v5.10 so the CI matrix
actually covers the full declared support range.

Closes #16

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@atoomic

atoomic commented Apr 30, 2026

Copy link
Copy Markdown
Owner

@Koan-Bot rebase

@Koan-Bot

Copy link
Copy Markdown
Collaborator Author

Rebase with requested adjustments

Branch koan.atoomic/ci-test-perl-5.10 was rebased onto master and review feedback was applied.

Stats

1 file changed, 1 insertion(+), 1 deletion(-)
Actions performed
  • Already-solved check: negative (confidence=high, reasoning=No commit on master changes the since-perl value from v5.14 to v5.10; commit 7a76fb4 introduced the )
  • Warning: pending (unsubmitted) review comments detected
  • Rebased koan.atoomic/ci-test-perl-5.10 onto origin/master
  • Pre-push CI check: previous run passed
  • Force-pushed koan.atoomic/ci-test-perl-5.10 to origin
  • CI check enqueued in ## CI (async)

CI status

CI will be checked asynchronously.


Automated by Kōan

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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