On 2023-10-06 Fr 10:07, Tom Lane wrote:
Peter Eisentraut <pe...@eisentraut.org> writes:
I don't have a good sense of what you are trying to optimize for. If
it's the mainline build-on-every-commit type, then I wonder how many
commits would really be affected by this. Like, how many commits touch
only a README file. If it's for things like the cfbot, then I think the
time-triggered builds would be more frequent than new patch versions, so
I don't know if these kinds of optimizations would affect anything.
As a quick cross-check, I searched our commit log to see how many
README-only commits there were so far this year. I found 11 since
January. (Several were triggered by the latest round of pgindent
code and process changes, so maybe this is more than typical.)
Not sure what that tells us about the value of changing the CI
logic, but it does seem like it could be worth the one-liner
change needed to teach buildfarm animals to ignore READMEs.
- trigger_exclude => qr[^doc/|\.po$],
+ trigger_exclude => qr[^doc/|/README$|\.po$],
I've put that in the sample config file for the next release.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
