Skip to content

Commit c7d193f

Browse files
robherleyrobherley
authored
Merge pull request #466 from actions/v4-beta
upload-artifact@v4
2 parents a8a3f3a + 13131bb commit c7d193f

File tree

132 files changed

+135223
-9604
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

132 files changed

+135223
-9604
lines changed

‎.github/workflows/check-dist.yml

+20-19
Original file line numberDiff line numberDiff line change
@@ -10,43 +10,44 @@ on:
1010
push:
1111
branches:
1212
- main
13+
paths-ignore:
14+
- '**.md'
1315
pull_request:
14-
branches:
15-
- main
16+
paths-ignore:
17+
- '**.md'
1618
workflow_dispatch:
1719

1820
jobs:
1921
check-dist:
2022
runs-on: ubuntu-latest
2123

2224
steps:
23-
- uses: actions/checkout@v3
25+
- uses: actions/checkout@v4
2426

25-
- name: Setup Node 16
26-
uses: actions/setup-node@v3
27+
- name: Setup Node 20
28+
uses: actions/setup-node@v4
2729
with:
28-
node-version: 16.x
30+
node-version: 20.x
2931
cache: 'npm'
30-
32+
3133
- name: Install dependencies
3234
run: npm ci
3335

34-
- name: Move the committed index.js file
35-
run: mv dist/index.js /tmp
36-
37-
- name: Rebuild with tsc
36+
- name: Rebuild the dist/ directory
3837
run: npm run build
3938

40-
- name: Rebuild the index.js file
41-
run: npm run release
42-
43-
- name: Compare the expected and actual index.js files
44-
run: git diff --ignore-all-space dist/index.js /tmp/index.js
39+
- name: Compare the expected and actual dist/ directories
40+
run: |
41+
if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
42+
echo "Detected uncommitted changes after build. See status below:"
43+
git diff
44+
exit 1
45+
fi
4546
id: diff
4647

4748
# If index.js was different than expected, upload the expected version as an artifact
48-
- uses: actions/upload-artifact@v3
49+
- uses: actions/upload-artifact@v4-beta
4950
if: ${{ failure() && steps.diff.conclusion == 'failure' }}
5051
with:
51-
name: index.js
52-
path: dist/index.js
52+
name: dist
53+
path: dist/

‎.github/workflows/licensed.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
runs-on: ubuntu-latest
1414
name: Check licenses
1515
steps:
16-
- uses: actions/checkout@v2
16+
- uses: actions/checkout@v4
1717
- run: npm ci
1818
- name: Install licensed
1919
run: |

‎.github/workflows/test.yml

+17-44
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
name: Test
2+
23
on:
34
push:
45
branches:
@@ -10,7 +11,6 @@ on:
1011
- '**.md'
1112

1213
jobs:
13-
1414
build:
1515
name: Build
1616

@@ -23,12 +23,12 @@ jobs:
2323

2424
steps:
2525
- name: Checkout
26-
uses: actions/checkout@v3
26+
uses: actions/checkout@v4
2727

28-
- name: Setup Node 16
29-
uses: actions/setup-node@v3
28+
- name: Setup Node 20
29+
uses: actions/setup-node@v4
3030
with:
31-
node-version: 16.x
31+
node-version: 20.x
3232
cache: 'npm'
3333

3434
- name: Install dependencies
@@ -54,43 +54,36 @@ jobs:
5454
mkdir -p path/to/dir-3
5555
echo "Lorem ipsum dolor sit amet" > path/to/dir-1/file1.txt
5656
echo "Hello world from file #2" > path/to/dir-2/file2.txt
57-
echo "This is a going to be a test for a large enough file that should get compressed with GZip. The @actions/artifact package uses GZip to upload files. This text should have a compression ratio greater than 100% so it should get uploaded using GZip" > path/to/dir-3/gzip.txt
5857
5958
# Upload a single file artifact
6059
- name: 'Upload artifact #1'
6160
uses: ./
6261
with:
63-
name: 'Artifact-A'
62+
name: 'Artifact-A-${{ matrix.runs-on }}'
6463
path: path/to/dir-1/file1.txt
6564

66-
# Upload using a wildcard pattern, name should default to 'artifact' if not provided
65+
# Upload using a wildcard pattern
6766
- name: 'Upload artifact #2'
6867
uses: ./
6968
with:
69+
name: 'Artifact-Wildcard-${{ matrix.runs-on }}'
7070
path: path/**/dir*/
7171

72-
# Upload a directory that contains a file that will be uploaded with GZip
72+
# Upload a multi-path artifact
7373
- name: 'Upload artifact #3'
7474
uses: ./
7575
with:
76-
name: 'GZip-Artifact'
77-
path: path/to/dir-3/
78-
79-
# Upload a directory that contains a file that will be uploaded with GZip
80-
- name: 'Upload artifact #4'
81-
uses: ./
82-
with:
83-
name: 'Multi-Path-Artifact'
76+
name: 'Multi-Path-Artifact-${{ matrix.runs-on }}'
8477
path: |
8578
path/to/dir-1/*
8679
path/to/dir-[23]/*
8780
!path/to/dir-3/*.txt
8881
8982
# Download Artifact #1 and verify the correctness of the content
9083
- name: 'Download artifact #1'
91-
uses: actions/download-artifact@v3
84+
uses: actions/download-artifact@v4-beta
9285
with:
93-
name: 'Artifact-A'
86+
name: 'Artifact-A-${{ matrix.runs-on }}'
9487
path: some/new/path
9588

9689
- name: 'Verify Artifact #1'
@@ -108,9 +101,9 @@ jobs:
108101

109102
# Download Artifact #2 and verify the correctness of the content
110103
- name: 'Download artifact #2'
111-
uses: actions/download-artifact@v3
104+
uses: actions/download-artifact@v4-beta
112105
with:
113-
name: 'artifact'
106+
name: 'Artifact-Wildcard-${{ matrix.runs-on }}'
114107
path: some/other/path
115108

116109
- name: 'Verify Artifact #2'
@@ -127,31 +120,11 @@ jobs:
127120
}
128121
shell: pwsh
129122

130-
# Download Artifact #3 and verify the correctness of the content
131-
- name: 'Download artifact #3'
132-
uses: actions/download-artifact@v3
133-
with:
134-
name: 'GZip-Artifact'
135-
path: gzip/artifact/path
136-
137-
# Because a directory was used as input during the upload the parent directories, path/to/dir-3/, should not be included in the uploaded artifact
138-
- name: 'Verify Artifact #3'
139-
run: |
140-
$gzipFile = "gzip/artifact/path/gzip.txt"
141-
if(!(Test-Path -path $gzipFile))
142-
{
143-
Write-Error "Expected file do not exist"
144-
}
145-
if(!((Get-Content $gzipFile) -ceq "This is a going to be a test for a large enough file that should get compressed with GZip. The @actions/artifact package uses GZip to upload files. This text should have a compression ratio greater than 100% so it should get uploaded using GZip"))
146-
{
147-
Write-Error "File contents of downloaded artifact is incorrect"
148-
}
149-
shell: pwsh
150-
123+
# Download Artifact #4 and verify the correctness of the content
151124
- name: 'Download artifact #4'
152-
uses: actions/download-artifact@v3
125+
uses: actions/download-artifact@v4-beta
153126
with:
154-
name: 'Multi-Path-Artifact'
127+
name: 'Multi-Path-Artifact-${{ matrix.runs-on }}'
155128
path: multi/artifact
156129

157130
- name: 'Verify Artifact #4'

‎.licenses/npm/@actions/artifact.dep.yml

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@actions/github.dep.yml

+20
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@actions/http-client.dep.yml

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@azure/abort-controller.dep.yml

+32
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@azure/core-auth.dep.yml

+33
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@azure/core-http.dep.yml

+33
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎.licenses/npm/@azure/core-lro.dep.yml

+33
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)