Re: [lng-odp] Github shows commits in wrong order

2017-11-20 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Savolainen, Petri (Nokia - FI/Espoo)
> Sent: Friday, November 17, 2017 10:24 AM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] Github shows commits in wrong order
> 
> Hi,
> 
> Here's evidence that Github shows sometimes commits in a wrong order. The
> first commit of this patch set is "api: std_types: add odp_percent_t data
> type", but it shows in the Github web page as the last one!
> 
> It's annoying and counterproductive that reviewers cannot trust on the
> patch order.
> 
> 
> -Petri

Github has done another spin on the commit order lottery. Now, the first patch 
("add odp_percent_t") is in the middle.

-Petri


Re: [lng-odp] Github shows commits in wrong order

2017-11-20 Thread Maxim Uvarov
created ticked for this on github.

On 20 November 2017 at 12:02, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia.com> wrote:

> > -Original Message-
> > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> > Savolainen, Petri (Nokia - FI/Espoo)
> > Sent: Friday, November 17, 2017 10:24 AM
> > To: lng-odp@lists.linaro.org
> > Subject: [lng-odp] Github shows commits in wrong order
> >
> > Hi,
> >
> > Here's evidence that Github shows sometimes commits in a wrong order. The
> > first commit of this patch set is "api: std_types: add odp_percent_t data
> > type", but it shows in the Github web page as the last one!
> >
> > It's annoying and counterproductive that reviewers cannot trust on the
> > patch order.
> >
> >
> > -Petri
>
> Github has done another spin on the commit order lottery. Now, the first
> patch ("add odp_percent_t") is in the middle.
>
> -Petri
>


Re: [lng-odp] Github shows commits in wrong order

2017-11-20 Thread Andriy Berestovskyy

Hey guys,
I guess it is a known GitHub "feature":

https://help.github.com/articles/why-are-my-commits-in-the-wrong-order/

Andriy

On 20.11.2017 10:15, Maxim Uvarov wrote:

created ticked for this on github.

On 20 November 2017 at 12:02, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia.com> wrote:


-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
Savolainen, Petri (Nokia - FI/Espoo)
Sent: Friday, November 17, 2017 10:24 AM
To: lng-odp@lists.linaro.org
Subject: [lng-odp] Github shows commits in wrong order

Hi,

Here's evidence that Github shows sometimes commits in a wrong order. The
first commit of this patch set is "api: std_types: add odp_percent_t data
type", but it shows in the Github web page as the last one!

It's annoying and counterproductive that reviewers cannot trust on the
patch order.


-Petri


Github has done another spin on the commit order lottery. Now, the first
patch ("add odp_percent_t") is in the middle.

-Petri



Re: [lng-odp] Github shows commits in wrong order

2017-11-20 Thread Maxim Uvarov
On 11/20/17 13:16, Andriy Berestovskyy wrote:
> Hey guys,
> I guess it is a known GitHub "feature":
> 
> https://help.github.com/articles/why-are-my-commits-in-the-wrong-order/
> 
> Andriy
> 

yep github team also replied with that link. And it's not clear if they
will fix it and when.

Maxim.

> On 20.11.2017 10:15, Maxim Uvarov wrote:
>> created ticked for this on github.
>>
>> On 20 November 2017 at 12:02, Savolainen, Petri (Nokia - FI/Espoo) <
>> petri.savolai...@nokia.com> wrote:
>>
>>>> -Original Message-
>>>> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
>>>> Savolainen, Petri (Nokia - FI/Espoo)
>>>> Sent: Friday, November 17, 2017 10:24 AM
>>>> To: lng-odp@lists.linaro.org
>>>> Subject: [lng-odp] Github shows commits in wrong order
>>>>
>>>> Hi,
>>>>
>>>> Here's evidence that Github shows sometimes commits in a wrong
>>>> order. The
>>>> first commit of this patch set is "api: std_types: add odp_percent_t
>>>> data
>>>> type", but it shows in the Github web page as the last one!
>>>>
>>>> It's annoying and counterproductive that reviewers cannot trust on the
>>>> patch order.
>>>>
>>>>
>>>> -Petri
>>>
>>> Github has done another spin on the commit order lottery. Now, the first
>>> patch ("add odp_percent_t") is in the middle.
>>>
>>> -Petri
>>>