Skip to content

Commit

Permalink
ci(github): use include in pr-modification.yaml (#12468)
Browse files Browse the repository at this point in the history
## Motivation

This was causing issues when there were no pr modified.

## Implementation information

 The indirection with include was hard to read so we removed it

Signed-off-by: Charly Molter <[email protected]>
  • Loading branch information
lahabana authored Jan 8, 2025
1 parent b6b11aa commit 85ce0a7
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/pr-modification.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
name: Get recent PRs
run: |
echo "out<<EOF" >> $GITHUB_OUTPUT
gh pr list -R ${{ github.repository }} --json number,title,url --search "updated:>=$(date --date='${{ env.LOOK_BACK }} ago' +'%Y-%m-%dT%H:%M:%S%z') -author:dependabot" | jq -c '{include: .}'>> $GITHUB_OUTPUT
gh pr list -R ${{ github.repository }} --json number,title,url --search "updated:>=$(date --date='${{ env.LOOK_BACK }} ago' +'%Y-%m-%dT%H:%M:%S%z') -author:dependabot" >> $GITHUB_OUTPUT
echo "EOF" >> $GITHUB_OUTPUT
- name: Show outputs
env:
Expand All @@ -49,7 +49,8 @@ jobs:
runs-on: ubuntu-24.04
if: needs.find-prs.outputs.recent_prs != '[]'
strategy:
matrix: ${{ fromJson(needs.find-prs.outputs.recent_prs) }}
matrix:
include: ${{ fromJson(needs.find-prs.outputs.recent_prs) }}
max-parallel: 1 # to avoid using too many runners
permissions:
pull-requests: write
Expand Down

0 comments on commit 85ce0a7

Please sign in to comment.