Skip to content

Commit 75f15fb

Browse files
committed
chore: collect demo data with npm-ls args
Signed-off-by: Jan Kowalleck <[email protected]>
1 parent 8ed0ae7 commit 75f15fb

File tree

1 file changed

+25
-8
lines changed

1 file changed

+25
-8
lines changed

.github/workflows/npm-ls_demo-results.yml

Lines changed: 25 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ env:
1313

1414
jobs:
1515
demo-results:
16-
name: '${{ matrix.subject }} npm${{ matrix.npm-version }} node${{ matrix.node-version }} ${{ matrix.os }}'
16+
name: '${{ matrix.subject }} ${{matrix.additional_nom-ls_args}} | npm${{ matrix.npm-version }} node${{ matrix.node-version }} ${{ matrix.os }}'
1717
runs-on: ${{ matrix.os }}
1818
strategy:
1919
fail-fast: false # gather all the results !
@@ -24,16 +24,17 @@ jobs:
2424
# - bundled-dependencies
2525
# - dev-dependencies
2626
# - juice-shop
27-
- local-dependencies
27+
# - local-dependencies
2828
- local-workspaces
2929
# - package-integrity
3030
# - package-with-build-id
31+
additional_nom-ls_args: [ '' ]
3132
npm-version:
3233
## see https://www.npmjs.com/package/npm?activeTab=versions
3334
## see also: https://github.com/npm/cli/releases
34-
- '10' # https://github.com/npm/cli/releases/tag/v10.0.0-pre.0
35-
- '9' # Current
36-
- '8' # Latest
35+
- '10' # Current
36+
- '9' # Legacy
37+
- '8' # Legacy
3738
- '7' # Legacy
3839
- '6' # Legacy
3940
node-version:
@@ -48,6 +49,22 @@ jobs:
4849
- ubuntu-latest
4950
- windows-latest
5051
- macos-latest
52+
include:
53+
- subject: local-workspaces
54+
additional_nom-ls_args: '-w my-local-e'
55+
npm-version: '10' # Current
56+
node-version: '22' # Current
57+
os: ubuntu-latest
58+
- subject: local-workspaces
59+
additional_nom-ls_args: '-w my-local -w my-local-e'
60+
npm-version: '10' # Current
61+
node-version: '22' # Current
62+
os: ubuntu-latest
63+
- subject: local-workspaces
64+
additional_nom-ls_args: '-ws'
65+
npm-version: '10' # Current
66+
node-version: '22' # Current
67+
os: ubuntu-latest
5168
exclude:
5269
- # macos-latest no longer supports node14
5370
os: macos-latest
@@ -81,7 +98,7 @@ jobs:
8198
- name: prepare output
8299
run: |
83100
mkdir -p '${{ env.RESULTS_DIR }}'
84-
echo 'OUT_FILE_PREFIX=${{ env.RESULTS_DIR }}/npm-ls_npm${{ matrix.npm-version }}_node${{ matrix.node-version }}_${{ matrix.os }}' >> $GITHUB_ENV
101+
echo 'OUT_FILE_PREFIX=${{ env.RESULTS_DIR }}/npm-ls${{ matrix.additional_nom-ls_args }}_npm${{ matrix.npm-version }}_node${{ matrix.node-version }}_${{ matrix.os }}' >> $GITHUB_ENV
85102
working-directory: '${{ env.DEMO_ROOT_DIR}}/${{ matrix.subject }}'
86103
shell: bash # don't want to write tht code twice, windows and unix-like
87104
- name: Setup Node.js ${{ matrix.node-version }}
@@ -103,15 +120,15 @@ jobs:
103120
# important: the command in use is the same used by the internal builders of this project
104121
run: >
105122
npm --prefix project
106-
ls --json --all --long
123+
ls --json --all --long ${{ matrix.additional_nom-ls_args }}
107124
2> ${{ env.OUT_FILE_PREFIX }}.err.log
108125
1> ${{ env.OUT_FILE_PREFIX }}.json
109126
working-directory: '${{ env.DEMO_ROOT_DIR}}/${{ matrix.subject }}'
110127
- name: Artifact RESULTS
111128
# see https://github.com/actions/upload-artifact
112129
uses: actions/upload-artifact@v4
113130
with:
114-
name: '${{ env.RESULTS_ARTIFACT }}_${{ matrix.subject }}_npm${{ matrix.npm-version }}_node${{ matrix.node-version }}_${{ matrix.os }}'
131+
name: '${{ env.RESULTS_ARTIFACT }}_${{ matrix.subject }}${{ matrix.additional_nom-ls_args }}_npm${{ matrix.npm-version }}_node${{ matrix.node-version }}_${{ matrix.os }}'
115132
path: '${{ env.DEMO_ROOT_DIR}}/*/${{ env.RESULTS_DIR }}'
116133
if-no-files-found: error
117134
retention-days: 3

0 commit comments

Comments
 (0)