Fix

basnijholt · Apr 6, 2024 · 976b596 · 976b596
1 parent 68ce944
commit 976b596
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/scripts/install_ha b/scripts/install_ha
@@ -1,12 +1,14 @@
 #!/usr/bin/env bash
-set -e
+set -ex
 cd "$(dirname "$0")/.."
 
 pip install -r core/requirements.txt --use-pep517
 
 # Older HA versions still have `codacov` in `requirements_test.txt`
 # however it is removed from PyPI, so we cannot install it
-sed -i '/codecov/d' core/requirements_test.txt
+if grep -q 'codecov' core/requirements_test.txt; then
+    sed -i '/codecov/d' core/requirements_test.txt
+fi
 pip install -r core/requirements_test.txt --use-pep517
 
 pip install -e core/ --use-pep517

diff --git a/scripts/setup-devcontainer b/scripts/setup-devcontainer
@@ -7,6 +7,5 @@ if [[ ! -d "core" ]]; then
     git clone --depth 1 --branch dev https://github.com/home-assistant/core.git
 fi
 
-chmod u+w core/
 ./scripts/install_ha
 pre-commit install-hooks