Hi Ill add it to the next PSC agenda - we can get flamed there instead :-P
Regards Tim On Thu, Jun 25, 2026 at 10:19 PM Nyall Dawson via QGIS-Developer < [email protected]> wrote: > > > On Thu, 25 Jun 2026 at 21:39, Even Rouault <[email protected]> > wrote: > > > > Hi Nyall, > > > > your answer doesn't really answer my question :-) Is the restriction for > every committer or a subset? > > > > Who has rights for changing qgis/qgis github settings? (not necessarily > asking for them to be extended, but just so we know). It would be good to > communicate when some settings that might affect workflows are adjusted > > We're definitely overdue for an overhaul on our github rights. Our current > policy is LAUGHABLY ridiculous (basically "you get merge rights once, > you've got them for life... even if you go off to work for ESRI or > disappear off the face of the earth" 🤣). > > It's not a discussion I feel like leading though. I tried once, and just > got flamed for even bringing it up. > > Nyall > > > > > > > > > I'd say that we should probably give up for now to auto-merge if it > blocks approved pull requests from being merged > > > > Even > > > > Le 25/06/2026 à 13:30, Nyall Dawson a écrit : > > > > > > > > On Thu, 25 June 2026, 9:24 pm Even Rouault via QGIS-Developer, < > [email protected]> wrote: > >> > >> Hi Julien, > >> > >> I just tried looking at your PR [0] and I also don't seem to have the > >> right to force merge it (only "auto-merge" rights that depend on green > CI) > >> > >> It would be indeed great to have some clarification of who might have > >> such rights or not, so we can see if there isn't a bottleneck that must > >> be resolved. I'm not sure why we would want such rights to be restricted > >> to a subset of people with "regular" merge rights rather than the whole > >> group. > > > > > > An alternative would be to remove the merge rules which require tests to > pass -- that'd avoid expanding rights while still allowing the blockage to > be removed. > > > > (Downside is that doing this breaks the ability to "auto merge") > > > > Nyall > >> > >> > >> Even > >> > >> Le 25/06/2026 à 12:24, Julien Cabieces via QGIS-Developer a écrit : > >> > Hi devs, > >> > > >> > I cannot merge PR #66483 [0] because there is at least one unrelated > >> > test failure every time the CI is launched (I restarted it 7 times), > although the PR was approved > >> > 6 days ago. > >> > > >> > I have already tackled the issue of the test_gui_fileuploader [1] > which was > >> > failing very often recently because of badssl.com being unstable but > there > >> > are now other issues. > >> > > >> > A few months ago, I had the rights to merge a PR even if tests were > >> > failing, but I don't have them anymore and I cannot contribute anymore > >> > on #66483 related issues (I have more fix commits based on this PR). > >> > > >> > It seems to me that other code committers still have these rights as > I see > >> > PRs with a red CI still being merged. > >> > > >> > May I have these rights too so I can continue to contribute normally? > >> > > >> > Shall every core committer have these rights ? > >> > > >> > Kind regards, > >> > Julien > >> > > >> > [0] https://github.com/qgis/QGIS/pull/66483 > >> > [1] https://github.com/qgis/QGIS/pull/66543 > >> > > >> -- > >> http://www.spatialys.com > >> My software is free, but my time generally not. > >> LLMs contribute to global warming and brain rot > >> > >> _______________________________________________ > >> QGIS-Developer mailing list > >> [email protected] > >> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer > >> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer > > > > -- > > http://www.spatialys.com > > My software is free, but my time generally not. > > LLMs contribute to global warming and brain rot > _______________________________________________ > QGIS-Developer mailing list > [email protected] > List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer > Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer > -- Tim Sutton *Kartoza Cofounder*Tim is a member of the QGIS Project Steering Committee *T *: +27(0) 87 809 2702 *E *: [email protected] *W* : kartoza.com *This email and any attachments are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you * *have received this email in error, please notify the sender immediately and delete it from your system. Unauthorised use, disclosure, or copying* *of the contents is prohibited.*
_______________________________________________ QGIS-Developer mailing list [email protected] List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
