Skip to content

Commit c25edee

Browse files
anderskgitster
authored andcommitted
receive-pack: lowercase error messages
Documentation/CodingGuidelines says “do not end error messages with a full stop” and “do not capitalize the first word”. Clean up existing messages, some of which we will be touching in later steps in the series, that deviate from these rules in this file, as a preparation for the main part of the topic. Signed-off-by: Anders Kaseorg <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 66996be commit c25edee

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

builtin/receive-pack.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ static int receive_pack_config(const char *var, const char *value, void *cb)
175175
strbuf_addf(&fsck_msg_types, "%c%s=%s",
176176
fsck_msg_types.len ? ',' : '=', var, value);
177177
else
178-
warning("Skipping unknown msg id '%s'", var);
178+
warning("skipping unknown msg id '%s'", var);
179179
return 0;
180180
}
181181

@@ -1589,9 +1589,9 @@ static const char *update(struct command *cmd, struct shallow_info *si)
15891589
if (!parse_object(the_repository, old_oid)) {
15901590
old_oid = NULL;
15911591
if (ref_exists(name)) {
1592-
rp_warning("Allowing deletion of corrupt ref.");
1592+
rp_warning("allowing deletion of corrupt ref");
15931593
} else {
1594-
rp_warning("Deleting a non-existent ref.");
1594+
rp_warning("deleting a non-existent ref");
15951595
cmd->did_not_exist = 1;
15961596
}
15971597
}
@@ -2490,9 +2490,9 @@ int cmd_receive_pack(int argc, const char **argv, const char *prefix)
24902490
argc = parse_options(argc, argv, prefix, options, receive_pack_usage, 0);
24912491

24922492
if (argc > 1)
2493-
usage_msg_opt(_("Too many arguments."), receive_pack_usage, options);
2493+
usage_msg_opt(_("too many arguments"), receive_pack_usage, options);
24942494
if (argc == 0)
2495-
usage_msg_opt(_("You must specify a directory."), receive_pack_usage, options);
2495+
usage_msg_opt(_("you must specify a directory"), receive_pack_usage, options);
24962496

24972497
service_dir = argv[0];
24982498

t/t5504-fetch-receive-strict.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ test_expect_success 'push with receive.fsck.missingEmail=warn' '
292292
receive.fsck.missingEmail warn &&
293293
git push --porcelain dst bogus >act 2>&1 &&
294294
grep "missingEmail" act &&
295-
test_i18ngrep "Skipping unknown msg id.*whatever" act &&
295+
test_i18ngrep "skipping unknown msg id.*whatever" act &&
296296
git --git-dir=dst/.git branch -D bogus &&
297297
git --git-dir=dst/.git config --add \
298298
receive.fsck.missingEmail ignore &&

0 commit comments

Comments
 (0)