@eht16 commented on this pull request.
> @@ -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 or view it on GitHub:
https://github.com/geany/geany/pull/3769#discussion_r1518925242
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/3769/review/1926618...@github.com>
