Fix: Ensure cross-platform compatibility for sed command in CI script

3 months ago · b1088344ff
parent d11ed2fb0a
commit b1088344ff
1 changed files with 5 additions and 1 deletions
--- a/scripts/checks/coverage.sh
+++ b/scripts/checks/coverage.sh
@ -12,7 +12,11 @@ if [ "${CI:-"false"}" == "true" ]; then
  # Foundry coverage
  forge coverage --report lcov --ir-minimum
  # Remove zero hits
-  sed -i '/,0/d' lcov.info
+  if [[ "$OSTYPE" == "darwin"* ]]; then
+    sed -i '' '/,0/d' lcov.info
+  else
+    sed -i '/,0/d' lcov.info
+  fi
 fi

 # Reports are then uploaded to Codecov automatically by workflow, and merged.