diff options
author | Luke Shumaker <lukeshu@datawire.io> | 2021-04-27 15:17:41 -0600 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-04-28 16:47:18 +0900 |
commit | 5cdae0f6fd76bd1c12304240a17a5a12632fbb12 (patch) | |
tree | 816c65eb84377a75bcdd7f09d1ee76bb342e6157 /t/t1430-bad-ref-name.sh | |
parent | subtree: remove duplicate check (diff) | |
download | tgif-5cdae0f6fd76bd1c12304240a17a5a12632fbb12.tar.xz |
subtree: add comments and sanity checks
For each function in subtree, add a usage comment saying what the
arguments are, and add an `assert` checking the number of arguments.
In figuring out each thing's arguments in order to write those comments
and assertions, it turns out that find_existing_splits is written as if
it takes multiple 'revs', but it is in fact only ever passed a single
'rev':
unrevs="$(find_existing_splits "$dir" "$rev")" || exit $?
So go ahead and codify that by documenting and asserting that it takes
exactly two arguments, one dir and one rev.
Signed-off-by: Luke Shumaker <lukeshu@datawire.io>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t1430-bad-ref-name.sh')
0 files changed, 0 insertions, 0 deletions