Skip to content

Commit b629258

Browse files
committed
ci: fix multi-version matrix failures on Perl < 5.24
The cpm dependency manager now requires Perl 5.24+, breaking the Install Dependencies step on all older Perl containers (5.8-5.22). Fix by removing the Install Dependencies step from the matrix job entirely — all runtime and test dependencies (XSLoader, Carp, Test::More) are core Perl modules. Also remove AUTHOR_TESTING and RELEASE_TESTING env vars since author tests (Test::Pod, Test::Pod::Coverage) are already covered by the ubuntu and disttest jobs.
1 parent 6c8b7f8 commit b629258

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

.github/workflows/testsuite.yml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,7 @@ jobs:
7575

7676
env:
7777
PERL_USE_UNSAFE_INC: 0
78-
AUTHOR_TESTING: 1
7978
AUTOMATED_TESTING: 1
80-
RELEASE_TESTING: 1
8179

8280
strategy:
8381
fail-fast: false
@@ -91,12 +89,6 @@ jobs:
9189
- uses: actions/checkout@v4
9290
- name: perl -V
9391
run: perl -V
94-
- name: Install Dependencies
95-
uses: perl-actions/install-with-cpm@v1
96-
with:
97-
sudo: false
98-
cpanfile: "cpanfile"
99-
args: "--with-develop"
10092
- run: perl Makefile.PL
10193
- run: make
10294
- run: make test

0 commit comments

Comments
 (0)