1 |
On Thu, 13 Oct 2016 14:12:17 +0200 |
2 |
Alexander Berntsen <bernalex@g.o> wrote: |
3 |
|
4 |
> Is there an associated bug for this? The patch itself looks reasonable |
5 |
> enough, but I prefer knowing the background for patches being deemed |
6 |
> necessary. |
7 |
> |
8 |
> |
9 |
> Also, get rid of the period in the commit message. And we seem to... |
10 |
> mostly... agree that it should be 'component: Capitalised |
11 |
> description', not 'component: lowercase description'. (No need to |
12 |
> email a v2 for something so trivial -- just fix it before pushing.) |
13 |
|
14 |
|
15 |
hmm, this one never got merged. |
16 |
|
17 |
I cleaned off the trailing period. |
18 |
|
19 |
Merged finally. |
20 |
-- |
21 |
Brian Dolbec <dolsen> |