Merge pull request #34698 from openedx/feanil/script_requirements
fix: Include script requirements in upgrade.
This commit is contained in:
@@ -28,7 +28,7 @@ jobs:
|
||||
echo $'Paths touched in PR:\n'"$paths"
|
||||
|
||||
# The ^"? is because git may quote weird file paths
|
||||
matched="$(echo "$paths" | grep -P '^"?requirements/' || true)"
|
||||
matched="$(echo "$paths" | grep -P '^"?((requirements/)|(scripts/.*?/requirements/))' || true)"
|
||||
echo $'Relevant paths:\n'"$matched"
|
||||
if [[ -n "$matched" ]]; then
|
||||
echo "RELEVANT=true" >> "$GITHUB_ENV"
|
||||
|
||||
@@ -86,7 +86,9 @@ jobs:
|
||||
with:
|
||||
branch: "${{ github.triggering_actor }}/upgrade-${{ inputs.package }}"
|
||||
branch-suffix: short-commit-hash
|
||||
add-paths: requirements
|
||||
add-paths: |
|
||||
requirements
|
||||
scripts/**/requirements
|
||||
commit-message: |
|
||||
feat: Upgrade Python dependency ${{ inputs.package }}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user