[gdal-dev] GH repo - write access
Even Rouault
even.rouault at spatialys.com
Sun Apr 8 14:01:59 PDT 2018
> Question: So _core_ GDAL developers (with write access) should also
> create PR? Than wait for feedback and any many cases apply PR on their
> own?
It is up to you, and your confidence in your change. There can be 3
strategies:
* directly push to origin (https://github.com/OSGeo gdal)
* do a pull request if Iyou want other people to have a chance to review and/
or have CI tests running
* or push to your own github account so that CI tests are still run on it, and
then merge if everything's OK
As far as I'm concerned, I do a mix of the 3 depending on the change, my mood
of the moment, etc...
If you directly push to origin, make sure to monitor
https://travis-ci.org/OSGeo/gdal/builds/
and
https://ci.appveyor.com/project/OSGeo/gdal/history
and be ready to take appropriate action if something breaks. There are
unfortunately some flakes sometimes, or timeouts, so a failure is not
necessarily yours, and you'll have to look at logs.
Even
--
Spatialys - Geospatial professional services
http://www.spatialys.com
More information about the gdal-dev
mailing list