Skip to content

Commit

Permalink
Box actions/upload-artifact and actions/download-artifact
Browse files Browse the repository at this point in the history
Occasionally these actions trigger back-off failures with the multiple
files.

Hoping there is no reason not to just zip proactively.

Hopefully none of the workflows care (obviously you cannot mix and match
versions of check-spelling within a single workflow for a single
artifact).

I do expect to encourage everyone to refresh their workflow with
0.0.20, but I don't think it will be strictly necessary.
  • Loading branch information
jsoref committed Jul 21, 2022
1 parent cdf0212 commit 1212521
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions unknown-words.sh
Original file line number Diff line number Diff line change
Expand Up @@ -732,6 +732,13 @@ define_variables() {
export early_warnings=$(mktemp)
if [ -n "$INPUT_INTERNAL_STATE_DIRECTORY" ]; then
data_dir="$INPUT_INTERNAL_STATE_DIRECTORY"
if [ -e "$data_dir/artifact.zip" ]; then
(
cd "$data_dir"
unzip -q 'artifact.zip'
rm artifact.zip
)
fi
else
data_dir=$(mktemp -d)
fi
Expand Down Expand Up @@ -1702,6 +1709,15 @@ quit() {
echo "$followup" > "$data_dir/followup"
echo "result_code=$status" >> "$GITHUB_ENV"
cat $output_variables
if ls "$data_dir" | grep -q .; then
artifact=$(mktemp)
(
cd "$data_dir"
zip -q "$artifact.zip" *
rm *
mv "$artifact.zip" 'artifact.zip'
)
fi
if to_boolean "$quit_without_error"; then
exit
fi
Expand Down

0 comments on commit 1212521

Please sign in to comment.