Put "Reverts" in the commit template?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
24 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Put "Reverts" in the commit template?

Jacques Le Roux
Administrator
Hi,

In some cases we need to revert a commit done for a Jira after we discover it causes an issue. We have not yet other means that using the fix word.
I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it please you) word in the commit template for this reason.
Because it's a different thing than really fixing the initial issue reported in the Jira but it's sill related to it

What do you think?

Jacques

Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacques Le Roux
Administrator
Also what do we do when we fix an issue introduced by a still open improvement Jira?

I use "Improves" because even if it's a bug it the continuity of an improvement, and anyway we can't add unlimited cases in the commit template (I'm
sure we will find other edge cases)

Jacques


Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :

> Hi,
>
> In some cases we need to revert a commit done for a Jira after we discover it causes an issue. We have not yet other means that using the fix word.
> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it please you) word in the commit template for this reason.
> Because it's a different thing than really fixing the initial issue reported in the Jira but it's sill related to it
>
> What do you think?
>
> Jacques
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Michael Brohl-3
In reply to this post by Jacques Le Roux
+1

Michael


Am 18.09.16 um 11:19 schrieb Jacques Le Roux:

> Hi,
>
> In some cases we need to revert a commit done for a Jira after we
> discover it causes an issue. We have not yet other means that using
> the fix word.
> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
> please you) word in the commit template for this reason.
> Because it's a different thing than really fixing the initial issue
> reported in the Jira but it's sill related to it
>
> What do you think?
>
> Jacques
>


smime.p7s (5K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

taher
Sure let's add reverts, but can we also agree on a unified format? I still
see Jacques using present and others using past which makes the original
purpose of the unification unattainable (to automate reports). I also see
different levels of support for text width from different committers

On Sun, Sep 18, 2016 at 6:41 PM, Michael Brohl <[hidden email]>
wrote:

> +1
>
> Michael
>
>
> Am 18.09.16 um 11:19 schrieb Jacques Le Roux:
>
> Hi,
>>
>> In some cases we need to revert a commit done for a Jira after we
>> discover it causes an issue. We have not yet other means that using the fix
>> word.
>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>> please you) word in the commit template for this reason.
>> Because it's a different thing than really fixing the initial issue
>> reported in the Jira but it's sill related to it
>>
>> What do you think?
>>
>> Jacques
>>
>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacques Le Roux
Administrator
Le 18/09/2016 à 17:48, Taher Alkhateeb a écrit :
> Sure let's add reverts, but can we also agree on a unified format? I still
> see Jacques using present and others using past which makes the original
> purpose of the unification unattainable (to automate reports).

"(to automate reports)" :-o  Are you kidding?
We know it's (or should be) the 1st word on the 1st line before the 1st semicolon
Can't we use a regexp to parse on a token like eg "fix" for "Fixed, Fixes, Fix for". Note that the  last is normally the "norm"
Can't we really do what GitHub is doing in a similar case: https://help.github.com/articles/closing-issues-via-commit-messages/
I already offered my help multiple times but not even got an answer about my questions :/

Same for the parentheses around the Jira number they are superfluous. I see no reasons to put them there but finally unwillingly did :/
We know it's only (a) number/s (in case of multiple issues separated by a white space) on the second line preceded by an "OFBIZ-" string nothing hard
to parse

> I also see
> different levels of support for text width from different committers

I honestly thought this could help, but Jacopo disagreed http://markmail.org/message/twr7adj7wefz4dng
Anyway I did not yet get any answer from TortoiseSvn and did not find a way to have that automated in Eclipse commit feature (yes I commit a lot :p)
Do you have something easy to propose?

Thanks

Jacques

>
> On Sun, Sep 18, 2016 at 6:41 PM, Michael Brohl <[hidden email]>
> wrote:
>
>> +1
>>
>> Michael
>>
>>
>> Am 18.09.16 um 11:19 schrieb Jacques Le Roux:
>>
>> Hi,
>>> In some cases we need to revert a commit done for a Jira after we
>>> discover it causes an issue. We have not yet other means that using the fix
>>> word.
>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>> please you) word in the commit template for this reason.
>>> Because it's a different thing than really fixing the initial issue
>>> reported in the Jira but it's sill related to it
>>>
>>> What do you think?
>>>
>>> Jacques
>>>
>>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

taher
Hi Jacques,

I don't know what to say. What I understand is that you don't want to use
the template that committers are now abiding by and which you agreed on
yourself before saying "I changed my mind"! I don't remember having any
discussion beyond a few complaints inside your own commit messages.

There are numerous ways to setup your environment so you can commit more
easily, but it's hard to say how exactly for each committer's operating
system and set of tools. But I think it's not difficult with a proof being
that committers were getting things nicely committed during the community
day which had lots of commits.

The easiest part of committing code is the commit message, the hard work is
in checking that the patch is correct, and that all tests pass, and that
quality is good etc. So committing a lot without a good review is perhaps
detrimental to the code base, and if you commit so much and so fast that
you don't have time to add a few characters, words or new lines here and
there then that might indicate you're rushing, which might affect the
quality of your commits.

Anyway, I won't dwell more on this subject, it seems you already took a
decision.

Taher Alkhateeb

On Sun, Sep 18, 2016 at 8:33 PM, Jacques Le Roux <
[hidden email]> wrote:

> Le 18/09/2016 à 17:48, Taher Alkhateeb a écrit :
>
>> Sure let's add reverts, but can we also agree on a unified format? I still
>> see Jacques using present and others using past which makes the original
>> purpose of the unification unattainable (to automate reports).
>>
>
> "(to automate reports)" :-o  Are you kidding?
> We know it's (or should be) the 1st word on the 1st line before the 1st
> semicolon
> Can't we use a regexp to parse on a token like eg "fix" for "Fixed, Fixes,
> Fix for". Note that the  last is normally the "norm"
> Can't we really do what GitHub is doing in a similar case:
> https://help.github.com/articles/closing-issues-via-commit-messages/
> I already offered my help multiple times but not even got an answer about
> my questions :/
>
> Same for the parentheses around the Jira number they are superfluous. I
> see no reasons to put them there but finally unwillingly did :/
> We know it's only (a) number/s (in case of multiple issues separated by a
> white space) on the second line preceded by an "OFBIZ-" string nothing hard
> to parse
>
> I also see
>> different levels of support for text width from different committers
>>
>
> I honestly thought this could help, but Jacopo disagreed
> http://markmail.org/message/twr7adj7wefz4dng
> Anyway I did not yet get any answer from TortoiseSvn and did not find a
> way to have that automated in Eclipse commit feature (yes I commit a lot :p)
> Do you have something easy to propose?
>
> Thanks
>
> Jacques
>
>
>
>> On Sun, Sep 18, 2016 at 6:41 PM, Michael Brohl <[hidden email]>
>> wrote:
>>
>> +1
>>>
>>> Michael
>>>
>>>
>>> Am 18.09.16 um 11:19 schrieb Jacques Le Roux:
>>>
>>> Hi,
>>>
>>>> In some cases we need to revert a commit done for a Jira after we
>>>> discover it causes an issue. We have not yet other means that using the
>>>> fix
>>>> word.
>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>>> please you) word in the commit template for this reason.
>>>> Because it's a different thing than really fixing the initial issue
>>>> reported in the Jira but it's sill related to it
>>>>
>>>> What do you think?
>>>>
>>>> Jacques
>>>>
>>>>
>>>>
>>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacques Le Roux
Administrator
In reply to this post by Jacques Le Roux
Done

Jacques


Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :

> Hi,
>
> In some cases we need to revert a commit done for a Jira after we discover it causes an issue. We have not yet other means that using the fix word.
> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it please you) word in the commit template for this reason.
> Because it's a different thing than really fixing the initial issue reported in the Jira but it's sill related to it
>
> What do you think?
>
> Jacques
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacopo Cappellato-5
I have changed it to "Reverted" for consistency reasons.

Jacopo



On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
[hidden email]> wrote:

> Done
>
> Jacques
>
>
> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>
>> Hi,
>>
>> In some cases we need to revert a commit done for a Jira after we
>> discover it causes an issue. We have not yet other means that using the fix
>> word.
>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>> please you) word in the commit template for this reason.
>> Because it's a different thing than really fixing the initial issue
>> reported in the Jira but it's sill related to it
>>
>> What do you think?
>>
>> Jacques
>>
>>
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacques Le Roux
Administrator
Hi Jacopo,

What is the logical behind this? It's not the first time I ask and I'd really like to have a clarification.

We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?

Thanks

Jacques

Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :

> I have changed it to "Reverted" for consistency reasons.
>
> Jacopo
>
>
>
> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
> [hidden email]> wrote:
>
>> Done
>>
>> Jacques
>>
>>
>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>
>>> Hi,
>>>
>>> In some cases we need to revert a commit done for a Jira after we
>>> discover it causes an issue. We have not yet other means that using the fix
>>> word.
>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>> please you) word in the commit template for this reason.
>>> Because it's a different thing than really fixing the initial issue
>>> reported in the Jira but it's sill related to it
>>>
>>> What do you think?
>>>
>>> Jacques
>>>
>>>
>>>

Reply | Threaded
Open this post in threaded view
|

Re: Put "Reverts" in the commit template?

Jacopo Cappellato-5
I have answered you in the Wiki but quoting here for everyone's convenience:

"Jacques, for me it is a done deal!

As you suggests we could change:

   - Documentation --> Documented
   - Fix for --> Fixed

And the final list of verbs will be:

[Implemented|Improved|Fixed|Completed|Documented|Reverted]


If there are no objections (as I hope) we will finally have a template that
will be adopted by all committers (and also by Jacques)!"

On Thu, Sep 22, 2016 at 8:45 AM, Jacques Le Roux <
[hidden email]> wrote:

> Hi Jacopo,
>
> What is the logical behind this? It's not the first time I ask and I'd
> really like to have a clarification.
>
> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>
> Thanks
>
> Jacques
>
>
> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>
>> I have changed it to "Reverted" for consistency reasons.
>>
>> Jacopo
>>
>>
>>
>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>> [hidden email]> wrote:
>>
>> Done
>>>
>>> Jacques
>>>
>>>
>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>
>>> Hi,
>>>>
>>>> In some cases we need to revert a commit done for a Jira after we
>>>> discover it causes an issue. We have not yet other means that using the
>>>> fix
>>>> word.
>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>>> please you) word in the commit template for this reason.
>>>> Because it's a different thing than really fixing the initial issue
>>>> reported in the Jira but it's sill related to it
>>>>
>>>> What do you think?
>>>>
>>>> Jacques
>>>>
>>>>
>>>>
>>>>
>
Reply | Threaded
Open this post in threaded view
|

Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacques Le Roux
Administrator
In reply to this post by Jacques Le Roux
Jacopo,

I saw you answered on Confluence where I 1st asked
https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+commit+message+template?focusedCommentId=65871637#comment-65871637

Now, I understand that we need to pick a word, but why not being more flexible, similarly at what does GitHub
https://help.github.com/articles/closing-issues-via-commit-messages/ ?

I already suggested in previous threads that I could help if the process Michael uses to create the blog monthly report needs to be adapted.
In relation, I also created in the "Wiki page for the "monthly Jira issues list" creation in the blog" thread, without any answers so far :/

Thanks

Jacques


Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :

> Hi Jacopo,
>
> What is the logical behind this? It's not the first time I ask and I'd really like to have a clarification.
>
> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>
> Thanks
>
> Jacques
>
> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>> I have changed it to "Reverted" for consistency reasons.
>>
>> Jacopo
>>
>>
>>
>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>> [hidden email]> wrote:
>>
>>> Done
>>>
>>> Jacques
>>>
>>>
>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>
>>>> Hi,
>>>>
>>>> In some cases we need to revert a commit done for a Jira after we
>>>> discover it causes an issue. We have not yet other means that using the fix
>>>> word.
>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>>> please you) word in the commit template for this reason.
>>>> Because it's a different thing than really fixing the initial issue
>>>> reported in the Jira but it's sill related to it
>>>>
>>>> What do you think?
>>>>
>>>> Jacques
>>>>
>>>>
>>>>
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacopo Cappellato-5
On Thu, Sep 22, 2016 at 9:36 AM, Jacques Le Roux <
[hidden email]> wrote:

> Jacopo,
>
> I saw you answered on Confluence where I 1st asked
> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
> commit+message+template?focusedCommentId=65871637#comment-65871637
>
> Now, I understand that we need to pick a word, but why not being more
> flexible, similarly at what does GitHub https://help.github.com/articl
> es/closing-issues-via-commit-messages/ ?
>
>
It seems we agree with your first proposal of using "documented", "fixed",
"reverted" so I will update the Wiki now to reflect these changes; if the
other committers will object we will revisit this decision.

As regards the "flexibility" topic, in my opinion it will defeat the effort
of defining a template format: however I don't have further time to spend
on this topic at the moment so I will let others to continue it in this
thread and we will see what the outcome is.

Jacopo
Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Scott Gray-3
In reply to this post by Jacques Le Roux
I can't believe you're being so stubborn about something so minor Jacques,
it seems like very strange behavior to me.  For what it's worth as a native
English speaker, reading a commit message written in present-tense feels
very strange to me.  I'm looking at a history and reading something as
though it is current, it doesn't feel logical.

Regards
Scott

On 22 September 2016 at 19:36, Jacques Le Roux <[hidden email]
> wrote:

> Jacopo,
>
> I saw you answered on Confluence where I 1st asked
> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
> commit+message+template?focusedCommentId=65871637#comment-65871637
>
> Now, I understand that we need to pick a word, but why not being more
> flexible, similarly at what does GitHub https://help.github.com/articl
> es/closing-issues-via-commit-messages/ ?
>
> I already suggested in previous threads that I could help if the process
> Michael uses to create the blog monthly report needs to be adapted.
> In relation, I also created in the "Wiki page for the "monthly Jira issues
> list" creation in the blog" thread, without any answers so far :/
>
> Thanks
>
> Jacques
>
>
> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>
>> Hi Jacopo,
>>
>> What is the logical behind this? It's not the first time I ask and I'd
>> really like to have a clarification.
>>
>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>
>> Thanks
>>
>> Jacques
>>
>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>
>>> I have changed it to "Reverted" for consistency reasons.
>>>
>>> Jacopo
>>>
>>>
>>>
>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>> [hidden email]> wrote:
>>>
>>> Done
>>>>
>>>> Jacques
>>>>
>>>>
>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>
>>>> Hi,
>>>>>
>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>> discover it causes an issue. We have not yet other means that using
>>>>> the fix
>>>>> word.
>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>>>> please you) word in the commit template for this reason.
>>>>> Because it's a different thing than really fixing the initial issue
>>>>> reported in the Jira but it's sill related to it
>>>>>
>>>>> What do you think?
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>>
>>>>>
>>
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacques Le Roux
Administrator
Scott,

Reading your message I guess you did not read my previous explanation on why I prefer to use present instead of past. You may find more details in
digging in previous emails.

But long story short, I'm French so I can't compete in English with someone like you for who English is the mother tongue.

The reason I use present is because I got this habit while working with Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't
reveal anything saying he is from Southampton (at least he lives there). I was then used to use past also in commit messages. A habit I got while
seeing others committing in OFBiz. But when I saw Rupert  using present, it immediately made sense to me: at the moment you commit, you are doing an
action. So I should use present, I'm doing the commit, it's not yet done.

I don't know if Rupert will read or appreciate this message, but it's the truth!

Anyway I believe it's a moot point, and we should have the freedom to write as we prefer, like it's done in a successful project like GitHub...

Jacques


Le 22/09/2016 à 14:52, Scott Gray a écrit :

> I can't believe you're being so stubborn about something so minor Jacques,
> it seems like very strange behavior to me.  For what it's worth as a native
> English speaker, reading a commit message written in present-tense feels
> very strange to me.  I'm looking at a history and reading something as
> though it is current, it doesn't feel logical.
>
> Regards
> Scott
>
> On 22 September 2016 at 19:36, Jacques Le Roux <[hidden email]
>> wrote:
>> Jacopo,
>>
>> I saw you answered on Confluence where I 1st asked
>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>
>> Now, I understand that we need to pick a word, but why not being more
>> flexible, similarly at what does GitHub https://help.github.com/articl
>> es/closing-issues-via-commit-messages/ ?
>>
>> I already suggested in previous threads that I could help if the process
>> Michael uses to create the blog monthly report needs to be adapted.
>> In relation, I also created in the "Wiki page for the "monthly Jira issues
>> list" creation in the blog" thread, without any answers so far :/
>>
>> Thanks
>>
>> Jacques
>>
>>
>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>
>>> Hi Jacopo,
>>>
>>> What is the logical behind this? It's not the first time I ask and I'd
>>> really like to have a clarification.
>>>
>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>
>>>> I have changed it to "Reverted" for consistency reasons.
>>>>
>>>> Jacopo
>>>>
>>>>
>>>>
>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>> Done
>>>>> Jacques
>>>>>
>>>>>
>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>
>>>>> Hi,
>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>> discover it causes an issue. We have not yet other means that using
>>>>>> the fix
>>>>>> word.
>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as it
>>>>>> please you) word in the commit template for this reason.
>>>>>> Because it's a different thing than really fixing the initial issue
>>>>>> reported in the Jira but it's sill related to it
>>>>>>
>>>>>> What do you think?
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>

Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

rhowell
Hi yes, reading with interest, I agree with Jacques.
Commit messages should be Present Tense Imperative, Imperative Style.
There's plenty of links on Google as to why this is the widely adopted
industry standard.

On 22 September 2016 at 16:06, Jacques Le Roux <[hidden email]
> wrote:

> Scott,
>
> Reading your message I guess you did not read my previous explanation on
> why I prefer to use present instead of past. You may find more details in
> digging in previous emails.
>
> But long story short, I'm French so I can't compete in English with
> someone like you for who English is the mother tongue.
>
> The reason I use present is because I got this habit while working with
> Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't
> reveal anything saying he is from Southampton (at least he lives there). I
> was then used to use past also in commit messages. A habit I got while
> seeing others committing in OFBiz. But when I saw Rupert  using present, it
> immediately made sense to me: at the moment you commit, you are doing an
> action. So I should use present, I'm doing the commit, it's not yet done.
>
> I don't know if Rupert will read or appreciate this message, but it's the
> truth!
>
> Anyway I believe it's a moot point, and we should have the freedom to
> write as we prefer, like it's done in a successful project like GitHub...
>
> Jacques
>
>
> Le 22/09/2016 à 14:52, Scott Gray a écrit :
>
>> I can't believe you're being so stubborn about something so minor Jacques,
>> it seems like very strange behavior to me.  For what it's worth as a
>> native
>> English speaker, reading a commit message written in present-tense feels
>> very strange to me.  I'm looking at a history and reading something as
>> though it is current, it doesn't feel logical.
>>
>> Regards
>> Scott
>>
>> On 22 September 2016 at 19:36, Jacques Le Roux <
>> [hidden email]
>>
>>> wrote:
>>> Jacopo,
>>>
>>> I saw you answered on Confluence where I 1st asked
>>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>>
>>> Now, I understand that we need to pick a word, but why not being more
>>> flexible, similarly at what does GitHub https://help.github.com/articl
>>> es/closing-issues-via-commit-messages/ ?
>>>
>>> I already suggested in previous threads that I could help if the process
>>> Michael uses to create the blog monthly report needs to be adapted.
>>> In relation, I also created in the "Wiki page for the "monthly Jira
>>> issues
>>> list" creation in the blog" thread, without any answers so far :/
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>>
>>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>>
>>> Hi Jacopo,
>>>>
>>>> What is the logical behind this? It's not the first time I ask and I'd
>>>> really like to have a clarification.
>>>>
>>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>>
>>>> Thanks
>>>>
>>>> Jacques
>>>>
>>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>>
>>>> I have changed it to "Reverted" for consistency reasons.
>>>>>
>>>>> Jacopo
>>>>>
>>>>>
>>>>>
>>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>>> [hidden email]> wrote:
>>>>>
>>>>> Done
>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>>
>>>>>> Hi,
>>>>>>
>>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>>> discover it causes an issue. We have not yet other means that using
>>>>>>> the fix
>>>>>>> word.
>>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as
>>>>>>> it
>>>>>>> please you) word in the commit template for this reason.
>>>>>>> Because it's a different thing than really fixing the initial issue
>>>>>>> reported in the Jira but it's sill related to it
>>>>>>>
>>>>>>> What do you think?
>>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>
>


--
Rupert Howell

Provolve Ltd
Front Office, Deale House, 16 Lavant Street, Petersfield, GU32 3EW, UK

t: 01730 267868 / m: 079 0968 5308
e:  [hidden email]
w: http://www.provolve.com
Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Michael Brohl-3
Hi Rupert, Jacques, all,

if I search Google for it, I find many different opinions. For example,
here is a viarant from the Git documentation
http://git.kernel.org/?p=git/git.git;a=blob;f=Documentation/SubmittingPatches;h=ece3c77482b3ff006b973f1ed90b708e26556862;hb=HEAD

"the body should provide a meaningful commit message, which:
- uses the imperative, present tense: "change", not "changed" or "changes"."

All in all, we simply want to achieve something UNIFIED and Jacques is
still the only one objecting to use the proposed format. Noone else did.

But if we can't agree on past/present or whatever tense, I have a
different proposal which might be acceptable by all and end this stupid
discussion:

What if we state what this issue is or covers: a Bug, an Improvement, a
Documentation etc.?

The template then would look like:

===

[Implementation|Improvement|Bug|Task|Documentation|Revert]: [Jira title|Free text]
[(OFBIZ-xxxx)]

[More detailed explanation of what has been done and what the fix achieves,
sideeffects etc.]

[Thanks:] [xxxx for ... and yyyy for]

===

I would be happy to change to this format if we can all agree to use the
same without exceptions.

I really wish to end this and appreciate your benevolent consideration.

Thanks,

Michael


Am 22.09.16 um 17:21 schrieb Rupert Howell:

> Hi yes, reading with interest, I agree with Jacques.
> Commit messages should be Present Tense Imperative, Imperative Style.
> There's plenty of links on Google as to why this is the widely adopted
> industry standard.
>
> On 22 September 2016 at 16:06, Jacques Le Roux <[hidden email]
>> wrote:
>> Scott,
>>
>> Reading your message I guess you did not read my previous explanation on
>> why I prefer to use present instead of past. You may find more details in
>> digging in previous emails.
>>
>> But long story short, I'm French so I can't compete in English with
>> someone like you for who English is the mother tongue.
>>
>> The reason I use present is because I got this habit while working with
>> Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't
>> reveal anything saying he is from Southampton (at least he lives there). I
>> was then used to use past also in commit messages. A habit I got while
>> seeing others committing in OFBiz. But when I saw Rupert  using present, it
>> immediately made sense to me: at the moment you commit, you are doing an
>> action. So I should use present, I'm doing the commit, it's not yet done.
>>
>> I don't know if Rupert will read or appreciate this message, but it's the
>> truth!
>>
>> Anyway I believe it's a moot point, and we should have the freedom to
>> write as we prefer, like it's done in a successful project like GitHub...
>>
>> Jacques
>>
>>
>> Le 22/09/2016 à 14:52, Scott Gray a écrit :
>>
>>> I can't believe you're being so stubborn about something so minor Jacques,
>>> it seems like very strange behavior to me.  For what it's worth as a
>>> native
>>> English speaker, reading a commit message written in present-tense feels
>>> very strange to me.  I'm looking at a history and reading something as
>>> though it is current, it doesn't feel logical.
>>>
>>> Regards
>>> Scott
>>>
>>> On 22 September 2016 at 19:36, Jacques Le Roux <
>>> [hidden email]
>>>
>>>> wrote:
>>>> Jacopo,
>>>>
>>>> I saw you answered on Confluence where I 1st asked
>>>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>>>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>>>
>>>> Now, I understand that we need to pick a word, but why not being more
>>>> flexible, similarly at what does GitHub https://help.github.com/articl
>>>> es/closing-issues-via-commit-messages/ ?
>>>>
>>>> I already suggested in previous threads that I could help if the process
>>>> Michael uses to create the blog monthly report needs to be adapted.
>>>> In relation, I also created in the "Wiki page for the "monthly Jira
>>>> issues
>>>> list" creation in the blog" thread, without any answers so far :/
>>>>
>>>> Thanks
>>>>
>>>> Jacques
>>>>
>>>>
>>>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>>>
>>>> Hi Jacopo,
>>>>> What is the logical behind this? It's not the first time I ask and I'd
>>>>> really like to have a clarification.
>>>>>
>>>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>>>
>>>>> Thanks
>>>>>
>>>>> Jacques
>>>>>
>>>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>>>
>>>>> I have changed it to "Reverted" for consistency reasons.
>>>>>> Jacopo
>>>>>>
>>>>>>
>>>>>>
>>>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>>>> [hidden email]> wrote:
>>>>>>
>>>>>> Done
>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>
>>>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>>>> discover it causes an issue. We have not yet other means that using
>>>>>>>> the fix
>>>>>>>> word.
>>>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as
>>>>>>>> it
>>>>>>>> please you) word in the commit template for this reason.
>>>>>>>> Because it's a different thing than really fixing the initial issue
>>>>>>>> reported in the Jira but it's sill related to it
>>>>>>>>
>>>>>>>> What do you think?
>>>>>>>>
>>>>>>>> Jacques
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>


smime.p7s (5K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacques Le Roux
Administrator
Michael,

Thanks for calling the conversation stupid, you could have refrained on this :/

For the rest I'm done, I tried to put a bit more of flexibility in this template, but since nobody cares (apart Rupert, thanks!), let it be. Now you
ALL must comply...

-0  (minus zero)

Jacques


Le 22/09/2016 à 18:06, Michael Brohl a écrit :

> Hi Rupert, Jacques, all,
>
> if I search Google for it, I find many different opinions. For example, here is a viarant from the Git documentation
> http://git.kernel.org/?p=git/git.git;a=blob;f=Documentation/SubmittingPatches;h=ece3c77482b3ff006b973f1ed90b708e26556862;hb=HEAD
>
> "the body should provide a meaningful commit message, which:
> - uses the imperative, present tense: "change", not "changed" or "changes"."
>
> All in all, we simply want to achieve something UNIFIED and Jacques is still the only one objecting to use the proposed format. Noone else did.
>
> But if we can't agree on past/present or whatever tense, I have a different proposal which might be acceptable by all and end this stupid discussion:
>
> What if we state what this issue is or covers: a Bug, an Improvement, a Documentation etc.?
>
> The template then would look like:
>
> ===
>
> [Implementation|Improvement|Bug|Task|Documentation|Revert]: [Jira title|Free text]
> [(OFBIZ-xxxx)]
>
> [More detailed explanation of what has been done and what the fix achieves,
> sideeffects etc.]
>
> [Thanks:] [xxxx for ... and yyyy for]
>
> ===
>
> I would be happy to change to this format if we can all agree to use the same without exceptions.
>
> I really wish to end this and appreciate your benevolent consideration.
>
> Thanks,
>
> Michael
>
>
> Am 22.09.16 um 17:21 schrieb Rupert Howell:
>> Hi yes, reading with interest, I agree with Jacques.
>> Commit messages should be Present Tense Imperative, Imperative Style.
>> There's plenty of links on Google as to why this is the widely adopted
>> industry standard.
>>
>> On 22 September 2016 at 16:06, Jacques Le Roux <[hidden email]
>>> wrote:
>>> Scott,
>>>
>>> Reading your message I guess you did not read my previous explanation on
>>> why I prefer to use present instead of past. You may find more details in
>>> digging in previous emails.
>>>
>>> But long story short, I'm French so I can't compete in English with
>>> someone like you for who English is the mother tongue.
>>>
>>> The reason I use present is because I got this habit while working with
>>> Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't
>>> reveal anything saying he is from Southampton (at least he lives there). I
>>> was then used to use past also in commit messages. A habit I got while
>>> seeing others committing in OFBiz. But when I saw Rupert using present, it
>>> immediately made sense to me: at the moment you commit, you are doing an
>>> action. So I should use present, I'm doing the commit, it's not yet done.
>>>
>>> I don't know if Rupert will read or appreciate this message, but it's the
>>> truth!
>>>
>>> Anyway I believe it's a moot point, and we should have the freedom to
>>> write as we prefer, like it's done in a successful project like GitHub...
>>>
>>> Jacques
>>>
>>>
>>> Le 22/09/2016 à 14:52, Scott Gray a écrit :
>>>
>>>> I can't believe you're being so stubborn about something so minor Jacques,
>>>> it seems like very strange behavior to me.  For what it's worth as a
>>>> native
>>>> English speaker, reading a commit message written in present-tense feels
>>>> very strange to me.  I'm looking at a history and reading something as
>>>> though it is current, it doesn't feel logical.
>>>>
>>>> Regards
>>>> Scott
>>>>
>>>> On 22 September 2016 at 19:36, Jacques Le Roux <
>>>> [hidden email]
>>>>
>>>>> wrote:
>>>>> Jacopo,
>>>>>
>>>>> I saw you answered on Confluence where I 1st asked
>>>>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>>>>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>>>>
>>>>> Now, I understand that we need to pick a word, but why not being more
>>>>> flexible, similarly at what does GitHub https://help.github.com/articl
>>>>> es/closing-issues-via-commit-messages/ ?
>>>>>
>>>>> I already suggested in previous threads that I could help if the process
>>>>> Michael uses to create the blog monthly report needs to be adapted.
>>>>> In relation, I also created in the "Wiki page for the "monthly Jira
>>>>> issues
>>>>> list" creation in the blog" thread, without any answers so far :/
>>>>>
>>>>> Thanks
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>>>>
>>>>> Hi Jacopo,
>>>>>> What is the logical behind this? It's not the first time I ask and I'd
>>>>>> really like to have a clarification.
>>>>>>
>>>>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>>>>
>>>>>> Thanks
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>>>>
>>>>>> I have changed it to "Reverted" for consistency reasons.
>>>>>>> Jacopo
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>>>>> [hidden email]> wrote:
>>>>>>>
>>>>>>> Done
>>>>>>>
>>>>>>>> Jacques
>>>>>>>>
>>>>>>>>
>>>>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>>>>> discover it causes an issue. We have not yet other means that using
>>>>>>>>> the fix
>>>>>>>>> word.
>>>>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as
>>>>>>>>> it
>>>>>>>>> please you) word in the commit template for this reason.
>>>>>>>>> Because it's a different thing than really fixing the initial issue
>>>>>>>>> reported in the Jira but it's sill related to it
>>>>>>>>>
>>>>>>>>> What do you think?
>>>>>>>>>
>>>>>>>>> Jacques
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacopo Cappellato-5
In reply to this post by rhowell
On Thu, Sep 22, 2016 at 5:21 PM, Rupert Howell <[hidden email]>
wrote:

> Hi yes, reading with interest, I agree with Jacques.
> Commit messages should be Present Tense Imperative, Imperative Style.
>

well, now I am a bit confused because Jacques is using Present Tense in
Third Person ("fixes") and not the Imperative Style ("fix")... but now I am
walking on the thin ice since English is not my mother tongue! From what I
understand the "present tense imperative" is the suggested style for Git
repositories... and this reminds me that we should start talking about
migrating from Svn to Git :-)

Jacopo

PS: Welcome back Rupert!
Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Shi Jinghai-3
In reply to this post by Jacques Le Roux
Ha, English, my favorite part. When I was 10, I learned my first 2 sentences of English:
1. Long life Chairman Mao!
2. Good morning comrade ... Gray

We are a worldwide community, please keep communication as simple as possible.

Good morning comrade everybody,

Shi Jinghai

-----邮件原件-----
发件人: Jacques Le Roux [mailto:[hidden email]]
发送时间: 2016年9月22日 23:07
收件人: [hidden email]
主题: Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Scott,

Reading your message I guess you did not read my previous explanation on why I prefer to use present instead of past. You may find more details in digging in previous emails.

But long story short, I'm French so I can't compete in English with someone like you for who English is the mother tongue.

The reason I use present is because I got this habit while working with Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't reveal anything saying he is from Southampton (at least he lives there). I was then used to use past also in commit messages. A habit I got while seeing others committing in OFBiz. But when I saw Rupert  using present, it immediately made sense to me: at the moment you commit, you are doing an action. So I should use present, I'm doing the commit, it's not yet done.

I don't know if Rupert will read or appreciate this message, but it's the truth!

Anyway I believe it's a moot point, and we should have the freedom to write as we prefer, like it's done in a successful project like GitHub...

Jacques


Le 22/09/2016 à 14:52, Scott Gray a écrit :

> I can't believe you're being so stubborn about something so minor
> Jacques, it seems like very strange behavior to me.  For what it's
> worth as a native English speaker, reading a commit message written in
> present-tense feels very strange to me.  I'm looking at a history and
> reading something as though it is current, it doesn't feel logical.
>
> Regards
> Scott
>
> On 22 September 2016 at 19:36, Jacques Le Roux
> <[hidden email]
>> wrote:
>> Jacopo,
>>
>> I saw you answered on Confluence where I 1st asked
>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>
>> Now, I understand that we need to pick a word, but why not being more
>> flexible, similarly at what does GitHub
>> https://help.github.com/articl es/closing-issues-via-commit-messages/ ?
>>
>> I already suggested in previous threads that I could help if the
>> process Michael uses to create the blog monthly report needs to be adapted.
>> In relation, I also created in the "Wiki page for the "monthly Jira
>> issues list" creation in the blog" thread, without any answers so far
>> :/
>>
>> Thanks
>>
>> Jacques
>>
>>
>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>
>>> Hi Jacopo,
>>>
>>> What is the logical behind this? It's not the first time I ask and
>>> I'd really like to have a clarification.
>>>
>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>
>>>> I have changed it to "Reverted" for consistency reasons.
>>>>
>>>> Jacopo
>>>>
>>>>
>>>>
>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>> Done
>>>>> Jacques
>>>>>
>>>>>
>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>
>>>>> Hi,
>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>> discover it causes an issue. We have not yet other means that
>>>>>> using the fix word.
>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted"
>>>>>> as it please you) word in the commit template for this reason.
>>>>>> Because it's a different thing than really fixing the initial
>>>>>> issue reported in the Jira but it's sill related to it
>>>>>>
>>>>>> What do you think?
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>

Reply | Threaded
Open this post in threaded view
|

Re: Commit template, more flexibility [was Re: Put "Reverts" in the commit template?]

Jacques Le Roux
Administrator
In reply to this post by Michael Brohl-3
Le 22/09/2016 à 18:06, Michael Brohl a écrit :
> Hi Rupert, Jacques, all,
>
> if I search Google for it, I find many different opinions. For example, here is a viarant from the Git documentation
> http://git.kernel.org/?p=git/git.git;a=blob;f=Documentation/SubmittingPatches;h=ece3c77482b3ff006b973f1ed90b708e26556862;hb=HEAD

Just to add that it's why I suggested to use verbs variations, like do GitHub. To allow users to use the one they prefer, but

>
> "the body should provide a meaningful commit message, which:
> - uses the imperative, present tense: "change", not "changed" or "changes"."
>
> All in all, we simply want to achieve something UNIFIED and Jacques is still the only one objecting to use the proposed format. Noone else did.
>
> But if we can't agree on past/present or whatever tense, I have a different proposal which might be acceptable by all and end this stupid discussion:
>
> What if we state what this issue is or covers: a Bug, an Improvement, a Documentation etc.?
>
> The template then would look like:
>
> ===
>
> [Implementation|Improvement|Bug|Task|Documentation|Revert]: [Jira title|Free text]
> [(OFBIZ-xxxx)]
>
> [More detailed explanation of what has been done and what the fix achieves,
> sideeffects etc.]
>
> [Thanks:] [xxxx for ... and yyyy for]
>
> ===
>
> I would be happy to change to this format if we can all agree to use the same without exceptions.

Revert then should be Reversion: http://www.etymonline.com/index.php?term=reversion

Jacques

>
> I really wish to end this and appreciate your benevolent consideration.
>
> Thanks,
>
> Michael
>
>
> Am 22.09.16 um 17:21 schrieb Rupert Howell:
>> Hi yes, reading with interest, I agree with Jacques.
>> Commit messages should be Present Tense Imperative, Imperative Style.
>> There's plenty of links on Google as to why this is the widely adopted
>> industry standard.
>>
>> On 22 September 2016 at 16:06, Jacques Le Roux <[hidden email]
>>> wrote:
>>> Scott,
>>>
>>> Reading your message I guess you did not read my previous explanation on
>>> why I prefer to use present instead of past. You may find more details in
>>> digging in previous emails.
>>>
>>> But long story short, I'm French so I can't compete in English with
>>> someone like you for who English is the mother tongue.
>>>
>>> The reason I use present is because I got this habit while working with
>>> Rupert Howell. You know, the guy who wrote the first OFBiz book. I don't
>>> reveal anything saying he is from Southampton (at least he lives there). I
>>> was then used to use past also in commit messages. A habit I got while
>>> seeing others committing in OFBiz. But when I saw Rupert using present, it
>>> immediately made sense to me: at the moment you commit, you are doing an
>>> action. So I should use present, I'm doing the commit, it's not yet done.
>>>
>>> I don't know if Rupert will read or appreciate this message, but it's the
>>> truth!
>>>
>>> Anyway I believe it's a moot point, and we should have the freedom to
>>> write as we prefer, like it's done in a successful project like GitHub...
>>>
>>> Jacques
>>>
>>>
>>> Le 22/09/2016 à 14:52, Scott Gray a écrit :
>>>
>>>> I can't believe you're being so stubborn about something so minor Jacques,
>>>> it seems like very strange behavior to me.  For what it's worth as a
>>>> native
>>>> English speaker, reading a commit message written in present-tense feels
>>>> very strange to me.  I'm looking at a history and reading something as
>>>> though it is current, it doesn't feel logical.
>>>>
>>>> Regards
>>>> Scott
>>>>
>>>> On 22 September 2016 at 19:36, Jacques Le Roux <
>>>> [hidden email]
>>>>
>>>>> wrote:
>>>>> Jacopo,
>>>>>
>>>>> I saw you answered on Confluence where I 1st asked
>>>>> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+
>>>>> commit+message+template?focusedCommentId=65871637#comment-65871637
>>>>>
>>>>> Now, I understand that we need to pick a word, but why not being more
>>>>> flexible, similarly at what does GitHub https://help.github.com/articl
>>>>> es/closing-issues-via-commit-messages/ ?
>>>>>
>>>>> I already suggested in previous threads that I could help if the process
>>>>> Michael uses to create the blog monthly report needs to be adapted.
>>>>> In relation, I also created in the "Wiki page for the "monthly Jira
>>>>> issues
>>>>> list" creation in the blog" thread, without any answers so far :/
>>>>>
>>>>> Thanks
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>> Le 22/09/2016 à 08:45, Jacques Le Roux a écrit :
>>>>>
>>>>> Hi Jacopo,
>>>>>> What is the logical behind this? It's not the first time I ask and I'd
>>>>>> really like to have a clarification.
>>>>>>
>>>>>> We have "Fix for" and "Documentation". Why not "Fixed" and "Documented"?
>>>>>>
>>>>>> Thanks
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>> Le 21/09/2016 à 19:09, Jacopo Cappellato a écrit :
>>>>>>
>>>>>> I have changed it to "Reverted" for consistency reasons.
>>>>>>> Jacopo
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Wed, Sep 21, 2016 at 7:01 PM, Jacques Le Roux <
>>>>>>> [hidden email]> wrote:
>>>>>>>
>>>>>>> Done
>>>>>>>
>>>>>>>> Jacques
>>>>>>>>
>>>>>>>>
>>>>>>>> Le 18/09/2016 à 11:19, Jacques Le Roux a écrit :
>>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>>> In some cases we need to revert a commit done for a Jira after we
>>>>>>>>> discover it causes an issue. We have not yet other means that using
>>>>>>>>> the fix
>>>>>>>>> word.
>>>>>>>>> I suggest we put in the "Reverts" (or "Revert for" or "Reverted" as
>>>>>>>>> it
>>>>>>>>> please you) word in the commit template for this reason.
>>>>>>>>> Because it's a different thing than really fixing the initial issue
>>>>>>>>> reported in the Jira but it's sill related to it
>>>>>>>>>
>>>>>>>>> What do you think?
>>>>>>>>>
>>>>>>>>> Jacques
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>
>
>

12