[Gluster-infra] [Bug 1431199] New: Request to automate closing github PRs
bugzilla at redhat.com
bugzilla at redhat.com
Fri Mar 10 15:07:10 UTC 2017
https://bugzilla.redhat.com/show_bug.cgi?id=1431199
Bug ID: 1431199
Summary: Request to automate closing github PRs
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Priority: medium
Assignee: bugs at gluster.org
Reporter: srangana at redhat.com
CC: bugs at gluster.org, gluster-infra at gluster.org
This request is discussed here:
http://lists.gluster.org/pipermail/maintainers/2017-March/002325.html
We do not accept github PRs as our work flow is gerrit based.
To educate contributors, one of the things we could do is to put up a
template as in [1] for the PULL_REQUEST_TEMPLATE. It will look like this
[2] when users attempt to submit PRs.
An alternate idea suggested by Nigel was to accept PRs, but use a
Jenkins job to auto close the same, with some generic (or very specific)
instructions on how to submit it to our gerrit instance.
The advantage of the latter method is that we do not lose the code
change suggested, and we can still evaluate the same and reach out to
the contributor and also possibly get the change into our gerrit instance.
The latter approach looks good, and we would like to have this inplemented.
This bug is being raised for the same.
Also, this is not an urgent requirement, this can happen in the next month or
beyond, as this does not change anything major for us at the moment.
[1] PR template:
https://review.gluster.org/#/c/16618/1/.github/PULL_REQUEST_TEMPLATE
[2] How a PR template appears to contributors:
https://goo.gl/photos/xxc8j3myqzD9tME98
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ONYxiJXfeZ&a=cc_unsubscribe
More information about the Gluster-infra
mailing list