[iTowns-dev] Video in repo

Augustin Trancart augustin.trancart at oslandia.com
Thu Jun 29 04:51:40 PDT 2017


I basically trust maintainer not to merge big files, but yes, if we can automate
it, it's even better.

Augustin Trancart - Oslandia
augustin.trancart at oslandia.com

On 29/06/2017 12:34, Thomas Broyer wrote:
> 
> 
> 2017-06-29 12:13 GMT+02:00 Augustin Trancart <augustin.trancart at oslandia.com
> <mailto:augustin.trancart at oslandia.com>>:
> 
>     Yes, agree with the idea of removing other big obsolete files as well.
> 
>     I'm not too worried about pull/pushes. If we require the files to be removed
>     before merge, these commits would become unreachable after the force-push, and
>     would eventually be garbage collected (providing we are careful at removing old
>     branches).
> 
> 
> Not sure I understand that last paragraph.
> My remark wrt rejecting pull requests (or pushes) with big files was to prevent
> this situation from happening again (in other words, as you said, "require the
> files to be removed before merge", but automate it).
> 
> Note also that GitHub will keep the files anyway in their special refs/pull/
> references [1], you'll just never sync them (unless you want it) because (by
> default) Git only syncs refs/heads/ and refs/tags/
> [1] https://help.github.com/articles/checking-out-pull-requests-locally/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.osgeo.org/pipermail/itowns-dev/attachments/20170629/02a1cc9b/attachment-0001.sig>


More information about the iTowns-dev mailing list