As proposed by @sanssecours we could add the "jenkins build all please" phrase to every build job.
This way we could avoid merging PRs that break some build job.
Maybe we should exclude the build jobs that are already triggered immediately anyway?
Any suggestions for a better name than "jenkins build all please"?
(see also #160)
Maybe we should exclude the build jobs that are already triggered immediately anyway?
Makes sense.
Any suggestions for a better name than "jenkins build all please"?
Nope. The phrase sounds good to me.
If we also want to exclude the build jobs that were executed by default, we might better have two phrases:
("build all" is still relevant if master is changed because this wont trigger the default build jobs for the PRs)
If we also want to exclude the build jobs that were executed by default, we might better have two phrases…
You are right. In that case I am for including the PR build jobs too. The simplification (only one special build phrase) should make up for the overhead of building the PR jobs twice.
I just added the new build phrase to the triggers of almost all jobs. I did not incorporate the phrase in the build jobs
elektra-git-buildpackage-jessie,elektra-git-buildpackage-stretch, andelektra-git-buildpackage-wheezy, since I am not sure that building the packages makes much sense in the case of a pull request. I also did not attach the phrase to
PR-checks, andelektra-todo. Do you think we should add some of the build jobs above to jenkins build all please too?
Thank you so much, its great to have jenkins build all please available. Can you document it in doc/GIT.md (if not done so already)?
Yes, the buildpackage jobs do not work for PRs.
I do not know what PR-checks is for, afaik it was created by @BernhardDenner
The elektra-todo is not really relevant (currently it is not used at all) but it also does not harm to add it? (One exception less.)
Thank you so much, its great to have
jenkins build all pleaseavailable. Can you document it indoc/GIT.md(if not done so already)?
I am already working on it.
I do not know what PR-checks is for, afaik it was created by @BernhardDenner
The
elektra-todois not really relevant (currently it is not used at all) but it also does not harm to add it? (One exception less.)
Okay. I just added the phrase to both jobs.