Merge pull request openedx#34698 from openedx/feanil/script_requirements

fix: Include script requirements in upgrade.
eduNEXT · May 6, 2024 · 28927da · 28927da
2 parents e6610f5 + b962383
commit 28927da
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/check-consistent-dependencies.yml b/.github/workflows/check-consistent-dependencies.yml
@@ -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"

diff --git a/.github/workflows/upgrade-one-python-dependency.yml b/.github/workflows/upgrade-one-python-dependency.yml
@@ -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 }}