diff options
| author | Jeff King <peff@peff.net> | 2022-12-14 11:19:43 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-12-15 09:13:56 +0900 |
| commit | a33d0fae76ab95e88d383793cac41934920296ba (patch) | |
| tree | 9c1dc67fa3d9938315d663d5629b87b71a89e560 /t/helper/test-submodule.c | |
| parent | ref-filter: reject arguments to %(HEAD) (diff) | |
| download | git-a33d0fae76ab95e88d383793cac41934920296ba.tar.gz git-a33d0fae76ab95e88d383793cac41934920296ba.zip | |
ref-filter: factor out "%(foo) does not take arguments" errors
Many atom parsers give the same error message, differing only in the
name of the atom. If we use "%s does not take arguments", that should
make life easier for translators, as they only need to translate one
string. And in doing so, we can easily pull it into a helper function to
make sure they are all using the exact same string.
I've added a basic test here for %(HEAD), just to make sure this code is
exercised at all in the test suite. We could cover each such atom, but
the effort-to-reward ratio of trying to maintain an exhaustive list
doesn't seem worth it.
Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/helper/test-submodule.c')
0 files changed, 0 insertions, 0 deletions