Skip to content

Commit

Permalink
Patch set-output deprecation in workflows (#9543)
Browse files Browse the repository at this point in the history
  • Loading branch information
matiboux authored Jan 3, 2024
1 parent 98c69b8 commit 1099c64
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/check-merge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,10 @@ jobs:
if: steps.blocked.outputs.result != 'true'
run: |
echo "Checking for changesets marked as minor"
echo '::set-output name=found::false'
echo "found=false" >> $GITHUB_OUTPUT
for file in ${{ steps.changed-files.outputs.all_changed_files }}; do
if grep -q "'astro': minor" "$file"; then
echo '::set-output name=found::true'
echo "found=true" >> $GITHUB_OUTPUT
echo "$file has a minor release tag"
fi
done
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,10 @@ jobs:
if [[ $output = '{ "changesets": [], "releases": [] }' ]]
then
echo 'No changeset found'
echo "::set-output name=run_job::true"
echo "run_job=true" >> $GITHUB_OUTPUT
else
echo 'changes found, push to latest skipped'
echo "::set-output name=run_job::false"
echo "run_job=false" >> $GITHUB_OUTPUT
fi
update:
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,9 @@ jobs:
- name: Generate Notification
id: notification
if: steps.changesets.outputs.published == 'true'
run: message=$(node scripts/notify/index.js '${{ steps.changesets.outputs.publishedPackages }}') && echo ::set-output name=message::${message//$'\n'/'%0A'}
run: |
message=$(node scripts/notify/index.js '${{ steps.changesets.outputs.publishedPackages }}')
echo "message=${message//$'\n'/'%0A'}" >> $GITHUB_OUTPUT
- name: Discord Notification
if: steps.changesets.outputs.published == 'true'
Expand Down

0 comments on commit 1099c64

Please sign in to comment.