On 27. 01. 21 13:45, Petr Menšík wrote:
I think one reason against maintainer's pull requests is poor tooling to
work with them. Filled fedpkg proposal to include ability to fork and
add personal fork to current repository or add when cloning [1].
I think current way discourages its use, because too many manual steps
have to be done before making PR.
1. https://bugzilla.redhat.com/show_bug.cgi?id=1920997
On 1/25/21 4:30 PM, Miro Hrončok wrote:
On 25. 01. 21 16:19, Stephen Gallagher wrote:
I'm fully in favor of this and I'd really like to see us add some
degree of CI gating to support it.
Note that unfortunately CI gating happens too late. It has no capability
to block commits that fail to build, because it only tests successful
builds and because it only tests already pushed changes. CI on Pull
Requests can solve this, but many packagers seem to be very much agianst
the idea of sending PRs to packages they maintain themselves :(
Is there already way to configure maintainer's fork branch to auto merge
and production build, once CI finishes successfully?
I think adding required steps into build process, where developer has to
watch for results and make manual steps is the problem. If I could just
push my change to my fork and let it autoprocess, I would use it when
not in hurry. But I demand first not involving multiple manual steps to
make production built from PR.
Could there be way as a maintainer to add a comment [build] to merged
PR, and pagure would start a new build? I think it does not matter who
started the build, but whose changes are included.
If I could just mark good looking changes and it would try to process
it, just notifying me whether successfully or not, I would use PR more
often. If I have to watch PR CI results myself and do manual steps
depending of its result, it discourages me. Would like some bot to do that.
There seems to be support for [citest] for retriggering CI on package.
Could something similar be used to autobuild PR of people with commit
rights on explicitly enabled branches?
Zuul can merge and build PRs automatically when CI passes.
https://fedoraproject.org/wiki/Zuul-based-ci
--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
