Skip to content

feat(ci): use supported Perl versions in CI#82

Merged
okurz merged 1 commit into
openSUSE:masterfrom
okurz:feature/poo203049_hashed_github_action
Jul 3, 2026
Merged

feat(ci): use supported Perl versions in CI#82
okurz merged 1 commit into
openSUSE:masterfrom
okurz:feature/poo203049_hashed_github_action

Conversation

@okurz

@okurz okurz commented Jul 2, 2026

Copy link
Copy Markdown
Member

Motivation:
Ensure CI workflows run tests against stable, supported Perl versions.

Design Choices:
Replace EOL Perl versions with 5.36, 5.38, 5.40, and latest. Update the
perltidy workflow to use a perl:5.40 container image.

Benefits:
Prevents installation failures on ancient Perl runtimes and verifies codebase
robustness against modern Perl interpreters.

@perlpunk perlpunk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which kind of problems do you see with older versions?
We discussed this in the past.
If the module officially supports perl 5.16, then we should be able to run tests for that version.
We should not only rely on cpantesters.

@okurz

okurz commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

Which kind of problems do you see with older versions? We discussed this in the past. If the module officially supports perl 5.16, then we should be able to run tests for that version. We should not only rely on cpantesters.

please see the problems in #81 . Maybe older Perl versions can still be tested with changes to the github action configuration.

@perlpunk

perlpunk commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Devel::Cover now requires perl 5.20.
Ideally we would be able to run coverage with a new version and test older perls without installing Devel::Cover.

@perlpunk

perlpunk commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

I'd say, test all versions starting with perl 5.20, that would be the easiest compromise.

Motivation:
Ensure CI workflows run tests against stable, supported Perl versions.

Design Choices:
Replace ancient, unsupported Perl versions with 5.20+ releases. Update
the setup step to conditionally install CI dependencies only on the latest
Perl version to prevent build and test runtime failures.

Benefits:
Ensures backward compatibility is tested on older supported Perl runtimes
while keeping CI setup fast, clean, and reliable.

Related issue: openSUSE#82
@okurz okurz force-pushed the feature/poo203049_hashed_github_action branch from a639801 to 44d715a Compare July 3, 2026 08:57
@okurz

okurz commented Jul 3, 2026

Copy link
Copy Markdown
Member Author

I'd say, test all versions starting with perl 5.20, that would be the easiest compromise.

done. Seems to work

@okurz okurz merged commit 5831e64 into openSUSE:master Jul 3, 2026
14 checks passed
@okurz okurz deleted the feature/poo203049_hashed_github_action branch July 3, 2026 18:41
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.

3 participants