Re: [PATCH] Add support for commit attributes
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Apr 10, 2014, 08:05 UTC
- Message-ID
- <CACsJy8Ddg99Foi89ntNQ7NB=J9qCH4hcXZ0maM7vxigfO8QVTQ@mail.gmail.com>
- In-Reply-To
- <CAFozjshu92OZbqWzcDZKN4v26Fb-7K=uJRYs4L41csrxfnSTUw@mail.gmail.com>
On Thu, Apr 10, 2014 at 1:27 PM, Diego Lago González <diego.lago.gonzalez@gmail.com> wrote:
Show 5 quoted lines
> Writing this data into the message, the user is forced to write it in the > correct format (I think is better to write key=value pairs as an option > instead of writing as message lines with spaces in key, between key and > equal sign and value, and other mistakes). And is simpler to parse these > attributes than the message itself.
the interpret-trailers series Christian Couder is cooking in 'pu' should handle this.
> And, what if the log message is seen from the command line instead of our CI > front-end? Why the CLI user (for example) should see information that does > not need or does not want to see?
which is why git-log (and all other porcelain commands) should learn to hide the value part (but not the key part).
-- Duy