Skip to content

Commit

Permalink
Fix pip_check (home-assistant#68421)
Browse files Browse the repository at this point in the history
  • Loading branch information
cdce8p authored Mar 20, 2022
1 parent 972afc5 commit 3b798ee
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .core_files.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ other: &other
requirements: &requirements
- .github/workflows/**
- homeassistant/package_constraints.txt
- script/pip_check
- requirements*.txt
- setup.cfg

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -686,7 +686,7 @@ jobs:
pip-check:
runs-on: ubuntu-latest
if: needs.changes.outputs.requirements == 'true'
if: needs.changes.outputs.requirements == 'true' || github.event.inputs.full == 'true'
needs:
- changes
- prepare-tests
Expand Down
3 changes: 2 additions & 1 deletion script/pip_check
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ PIP_CACHE=$1

# Number of existing dependency conflicts
# Update if a PR resolve one!
DEPENDENCY_CONFLICTS=4
DEPENDENCY_CONFLICTS=7

PIP_CHECK=$(pip check --cache-dir=$PIP_CACHE)
LINE_COUNT=$(echo "$PIP_CHECK" | wc -l)
Expand All @@ -14,6 +14,7 @@ then
echo "------"
echo "Requirements change added another dependency conflict."
echo "Make sure to check the 'pip check' output above!"
echo "Expected $DEPENDENCY_CONFLICTS conflicts, got $LINE_COUNT."
exit 1
elif [[ $((LINE_COUNT)) -lt $DEPENDENCY_CONFLICTS ]]
then
Expand Down

0 comments on commit 3b798ee

Please sign in to comment.