[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#47081] [PATCH v2 5/6] services: Remove mongodb service.
From: |
Christopher Baines |
Subject: |
[bug#47081] [PATCH v2 5/6] services: Remove mongodb service. |
Date: |
Sun, 14 Mar 2021 15:31:52 +0000 |
User-agent: |
mu4e 1.4.15; emacs 27.1 |
Léo Le Bouter <lle-bout@zaclys.net> writes:
> On Sun, 2021-03-14 at 14:54 +0000, Christopher Baines wrote:
>> I think it would be better to squash this commit in with the two
>> previous commits, so there just one commit to remove the service,
>> including the tests and documentation. That's a better "atomic"
>> change,
>> since all three bits (service itself, test and docs) are one thing in
>> my
>> opinion.
>
> Oh no, then all the individual commits and ordering work (so no single
> commit breaks GNU Guix or makes it inconsistent) is useless now :-(
I don't think there's going to be any problems with combining the
commits I'm suggesting. Given they're contiguous, there's no chance of
it breaking something, right?
> I will try squashing ASAP, but what do you recommend as title then? It
> seems to me this kind of violates GNU commit message guidelines.
I'm no expert at writing commit messages in the Guix/GNU style, but
maybe take a look at the commit that added the service [1]. I think
"services: Remove MongoDB." would be a reasonable title.
1:
https://git.savannah.gnu.org/cgit/guix.git/commit/?id=5266ff719e274056cb3e2b9740183f0063177255
signature.asc
Description: PGP signature
- [bug#47081] [PATCH 3/5] tests: databases: Remove mongodb test., (continued)
[bug#47081] [PATCH v2 6/6] gnu: Remove mongodb., Léo Le Bouter, 2021/03/11