@eht16 commented on this pull request.


In .github/workflows/build.yml:

> @@ -8,6 +8,13 @@ on:
   push:
     branches:
       - master
+    - '**'

I don't see why, at least for this case it seems a bit beyond the goal.

If we have use cases for excluding files from triggering a CI run, we
can add them but no excluding only two YAML files doesn't change much.

Without having checked it, I think about 95% of the commits will change files which should trigger the CI anyway, so don't waste resources maintaining an articial exclude list.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3769/review/1926618276@github.com>