From 08fc896cdff73c5f4e80f873cd2c05df7c0ab930 Mon Sep 17 00:00:00 2001 From: Olaf Alders Date: Thu, 13 Oct 2022 11:30:04 -0400 Subject: [PATCH 1/4] Don't reinstall perl on Windows --- .github/workflows/check.yml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/.github/workflows/check.yml b/.github/workflows/check.yml index a5ec85f..1ca0ac7 100644 --- a/.github/workflows/check.yml +++ b/.github/workflows/check.yml @@ -139,11 +139,6 @@ jobs: name: "windows" steps: - - name: Set up Perl - run: | - choco install strawberryperl - echo "##[add-path]C:\strawberry\c\bin;C:\strawberry\perl\site\bin;C:\strawberry\perl\bin" - - name: perl -V run: perl -V From 61cb2f4474dd499658750f0439d084f97a73d8a8 Mon Sep 17 00:00:00 2001 From: Olaf Alders Date: Thu, 13 Oct 2022 11:30:46 -0400 Subject: [PATCH 2/4] tidy YAML --- .github/workflows/check.yml | 41 +++++++++++++++++++------------------ 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/.github/workflows/check.yml b/.github/workflows/check.yml index 1ca0ac7..15abdd9 100644 --- a/.github/workflows/check.yml +++ b/.github/workflows/check.yml @@ -1,3 +1,4 @@ +--- name: check on: [push] @@ -5,7 +6,7 @@ on: [push] jobs: cpanm: runs-on: ubuntu-latest - name: "install cpanm" + name: 'install cpanm' steps: - uses: actions/checkout@v2 - name: uses install-with-cpanm @@ -19,13 +20,13 @@ jobs: one_module: runs-on: ubuntu-latest - name: "cpanm and a module" + name: 'cpanm and a module' steps: - uses: actions/checkout@v2 - name: uses install-with-cpanm uses: ./ with: - install: "Simple::Accessor" + install: 'Simple::Accessor' - run: perl -MSimple::Accessor -e1 ### ------------------------------------------------ @@ -34,7 +35,7 @@ jobs: multiple_modules: runs-on: ubuntu-latest - name: "cpanm & modules" + name: 'cpanm & modules' steps: - uses: actions/checkout@v2 - name: uses install-with-cpanm @@ -52,33 +53,33 @@ jobs: cpanfile_root: runs-on: ubuntu-latest - name: "cpanfile as root" + name: 'cpanfile as root' steps: - uses: actions/checkout@v2 - - name: "Create a cpanfile" + - name: 'Create a cpanfile' run: | echo "requires 'Simple::Accessor';" > cpanfile.test - name: uses install-with-cpanm uses: ./ with: - cpanfile: "cpanfile.test" + cpanfile: 'cpanfile.test' - run: perl -MSimple::Accessor -e1 cpanfile_nonroot: runs-on: ubuntu-latest - name: "cpanfile nonroot local::lib" + name: 'cpanfile nonroot local::lib' steps: - uses: actions/checkout@v2 - - name: "Create a cpanfile" + - name: 'Create a cpanfile' run: | echo "requires 'Simple::Accessor';" > cpanfile.test - name: uses install-with-cpanm uses: ./ with: - path: "cpanm-local" - cpanfile: "cpanfile.test" + path: 'cpanm-local' + cpanfile: 'cpanfile.test' sudo: false - args: "-L vendor" + args: '-L vendor' - run: sudo perl cpanm-local local::lib - run: perl -Mlocal::lib=--no-create,vendor -MSimple::Accessor -e1 @@ -88,15 +89,15 @@ jobs: with_tests: runs-on: ubuntu-latest - name: "install with tests" + name: 'install with tests' steps: - uses: actions/checkout@v2 - name: uses install-with-cpanm uses: ./ with: - install: "Simple::Accessor" + install: 'Simple::Accessor' tests: true - args: "-v" + args: '-v' - run: perl -MSimple::Accessor -e1 ### ------------------------------------------------ @@ -105,14 +106,14 @@ jobs: perl_tester: runs-on: ubuntu-latest - name: "perl v${{ matrix.perl-version }}" + name: 'perl v${{ matrix.perl-version }}' strategy: fail-fast: false matrix: perl-version: - - "5.30" - - "5.28" + - '5.30' + - '5.28' # ... container: @@ -136,14 +137,14 @@ jobs: ## ------------------------------------------------ windows: runs-on: windows-latest - name: "windows" + name: 'windows' steps: - name: perl -V run: perl -V - uses: actions/checkout@v2 - - name: "install-with-cpanm" + - name: 'install-with-cpanm' uses: ./ with: From a42c9c207d9e8525119db12969c5c4fa1fcc3416 Mon Sep 17 00:00:00 2001 From: Olaf Alders Date: Thu, 13 Oct 2022 11:31:15 -0400 Subject: [PATCH 3/4] Bump checkout action from v2 to v3 --- .github/workflows/check.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/check.yml b/.github/workflows/check.yml index 15abdd9..4e965c1 100644 --- a/.github/workflows/check.yml +++ b/.github/workflows/check.yml @@ -8,7 +8,7 @@ jobs: runs-on: ubuntu-latest name: 'install cpanm' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: uses install-with-cpanm uses: ./ - name: which cpanm @@ -22,7 +22,7 @@ jobs: runs-on: ubuntu-latest name: 'cpanm and a module' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: uses install-with-cpanm uses: ./ with: @@ -37,7 +37,7 @@ jobs: runs-on: ubuntu-latest name: 'cpanm & modules' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: uses install-with-cpanm uses: ./ with: @@ -55,7 +55,7 @@ jobs: runs-on: ubuntu-latest name: 'cpanfile as root' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: 'Create a cpanfile' run: | echo "requires 'Simple::Accessor';" > cpanfile.test @@ -69,7 +69,7 @@ jobs: runs-on: ubuntu-latest name: 'cpanfile nonroot local::lib' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: 'Create a cpanfile' run: | echo "requires 'Simple::Accessor';" > cpanfile.test @@ -91,7 +91,7 @@ jobs: runs-on: ubuntu-latest name: 'install with tests' steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: uses install-with-cpanm uses: ./ with: @@ -120,7 +120,7 @@ jobs: image: perldocker/perl-tester:${{ matrix.perl-version }} steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: uses install-with-cpanm uses: ./ with: @@ -143,7 +143,7 @@ jobs: - name: perl -V run: perl -V - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: 'install-with-cpanm' uses: ./ From d195ed4760575e7dc7daa39772e23f436e3a2934 Mon Sep 17 00:00:00 2001 From: Olaf Alders Date: Thu, 13 Oct 2022 13:40:01 -0400 Subject: [PATCH 4/4] Bump node12 to node16 in action.yml --- action.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/action.yml b/action.yml index 92c278a..e79ced1 100644 --- a/action.yml +++ b/action.yml @@ -43,5 +43,5 @@ inputs: default: "$Config{installsitescript}/cpanm" runs: - using: "node12" + using: "node16" main: "index.js"