[QGIS-Developer] Proposal for QGIS Documentation
senhor.neto at gmail.com
Wed Jan 16 03:12:04 PST 2019
Since most seem to agree with the bot implementation, can we do it in a way
- the PR owner (and other watchers) receive a message warning that the PR
is going to be closed with some time to act;
- Not automatically merging PRs, but warn the owner (and watchers) that the
owner (if with commit rights) can merge the PR because of not having active
reviews/comments for a while?
Can we also make possible to reactivate a PR once a new comment is done? I
am thinking about the new contributers that see their PR being closed and
have no way to Open it again. If they know no one, their work may be lost.
BTW, how much time of inactivity should trigger the bot? 2 weeks a warning,
3 weeks close?
Paolo Cavallini <cavallini at faunalia.it> escreveu no dia quarta, 16/01/2019
> In case nobody objects, I'd suggest implementing this soon, during HF at
> latest, better before.
> All the best.
> On 16/01/19 08:44, Andreas Neumann wrote:
> > Hi Matteo,
> > Your suggestion sounds good to me.
> > Andreas
> > Am 16.01.19 um 08:42 schrieb matteo:
> >> Hi Anita,
> >>> I am not very enthusiast of the automatic merging and closing of
> >>> PRs. People have different velocities, and I don't think it's good
> >>> to close a someones PR (that put some good work on it) just
> >>> no one with merge rights was able to review it. It is very
> >>> discouraging.
> >>> I think the suggestion was to automatically accept and merge them if
> >>> nobody with merge rights gets around to doing it.
> >> nope. What I'm suggesting is to automatic merge PR made by author having
> >> commit rights on the repo after a while and to close (or putting into
> >> stale) PR made by "external" people if there is not activity for a
> >> This could speed up the cue of PR and prevent eventual breaking of docs.
> >> Cheers
> >> Matteo
> >> _______________________________________________
> >> QGIS-Developer mailing list
> >> QGIS-Developer at lists.osgeo.org
> >> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> >> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> > _______________________________________________
> > QGIS-Developer mailing list
> > QGIS-Developer at lists.osgeo.org
> > List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> > Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Paolo Cavallini - www.faunalia.eu
> QGIS.ORG Chair:
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the QGIS-Developer