On Fri, 3 Jul 2020 at 14:28, Chris Angelico <ros...@gmail.com> wrote:

> On Fri, Jul 3, 2020 at 11:03 PM Ivan Pozdeev via Python-Dev
> <python-dev@python.org> wrote:
> >
> >
> > On 03.07.2020 15:26, Henk-Jaap Wagenaar wrote:
> >
> > On Fri, 3 Jul 2020 at 13:10, Ivan Pozdeev <v...@mail.mipt.ru> wrote:
> >>
> >> So what?
> >
> > Unnecessary
> >>
> >> They'll have to synchronise their history to ours to be able to make a
> PR. And if they don't, it doesn't matter for us what they do with the data
> anyway since they are responsible for maintaining it and keeping it
> relevant if they need to, not us
> >
> > That is not a very collaborative mindset.
> >
> >
> > I fail to see how. We provide all the tools to collaborate. If a person
> has a divergent history, they will see that when trying to collaborate
> (submit a PR or otherwise interact with our repo from theirs in any way)
> and will be able to fix that problem then and there.
> >
> >
> > Can somebody give an example of when we force-pushed before? Surely
> there should be a PEP outlining when we force push and how we communicate
> this to our "consumers" before/when we do so?
> >
>
> Even if someone isn't aware of the change, the PEPs repo *already*
> rewrites commits as they get merged,


That is not the right way of putting it in my opinion. What you describe
below is rewriting commits when merging/completing a pull request
(squashing is common). That is very different to going back to a commit
that is already in the same branch and rewriting that.


> so any discrepancies would be
> papered over cleanly. Consider:
>
> https://github.com/python/peps/pull/1488
> Two commits in the pull request
>
>
> https://github.com/python/peps/commit/045450aaf47941f3ee7daaa1774947b31885b2aa
> One commit in the final repo.
>
> If someone has the old version of the repo and creates a pull request,
> we'll just squash all the differences down and create a single commit
> that does the intent in a cleaner way. The only real effect will be a
> bit of noise during the PR process itself.
>
> There has ALREADY been far more hassle resulting from this commit
> message than there would be from a force-push.
>

Maybe, but rewriting it would (a) not make the "hassle" go away and (b)
would in my view create more "hassle".


>
> ChrisA
> _______________________________________________
> Python-Dev mailing list -- python-dev@python.org
> To unsubscribe send an email to python-dev-le...@python.org
> https://mail.python.org/mailman3/lists/python-dev.python.org/
> Message archived at
> https://mail.python.org/archives/list/python-dev@python.org/message/W3UOLWJZHRLJA75PJZ5O434FPPLBZMLQ/
> Code of Conduct: http://python.org/psf/codeofconduct/
>
_______________________________________________
Python-Dev mailing list -- python-dev@python.org
To unsubscribe send an email to python-dev-le...@python.org
https://mail.python.org/mailman3/lists/python-dev.python.org/
Message archived at 
https://mail.python.org/archives/list/python-dev@python.org/message/PG7U3HQOCQVMQFUQ5CPGGX5F4FIV3MHP/
Code of Conduct: http://python.org/psf/codeofconduct/

Reply via email to