<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/35a5e7388ad26224b787a1e64a965fa4b1095492">https://github.com/macports/macports-ports/commit/35a5e7388ad26224b787a1e64a965fa4b1095492</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 35a5e7388ad CI: don't report overall failure if port test fails
</span>35a5e7388ad is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 35a5e7388ad26224b787a1e64a965fa4b1095492
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Thu May 2 14:32:09 2024 +1000

<span style='display:block; white-space:pre;color:#404040;'>    CI: don't report overall failure if port test fails
</span>---
 .github/workflows/main.yml | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
</span><span style='display:block; white-space:pre;color:#808080;'>index 50d7dbb22fd..9ce73f59f0e 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/.github/workflows/main.yml
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/.github/workflows/main.yml
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -210,6 +210,7 @@ jobs:
</span> 
             echo "::group::Testing ${subport}"
 
<span style='display:block; white-space:pre;background:#e0ffe0;'>+            test_fail=0
</span>             set +e
             sudo mpbb \
               --work-dir "$workdir" \
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -225,8 +226,9 @@ jobs:
</span> 
               echo "❌ Tests failed, see the log for more details" >> $GITHUB_STEP_SUMMARY
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-              fail=1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-              continue
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+              # Not setting fail=1 as a 100% passing test suite is not considered
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+              # essential to merge a PR.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+              test_fail=1
</span>             fi
 
             test_msgs=$(port log --phase test --level warn "$subport")
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -241,7 +243,7 @@ jobs:
</span>               encoded_messages="port test ${subport}:%0A"
               encoded_messages+="$(echo "${test_msgs}" | sed -E 's/$/%0A/g' | tr -d '\n')"
               echo "::warning file=${path#${PWD}/ports/},line=1,col=1::${encoded_messages}"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            else
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            elif [ $test_fail -eq 0 ]; then
</span>                  echo "✅ Successfully tested" >> $GITHUB_STEP_SUMMARY
             fi
 
</pre><pre style='margin:0'>

</pre>