mbox series

[0/3] t1300: minor cleanups

Message ID 20210103093649.2221-1-martin.agren@gmail.com (mailing list archive)
Headers show
Series t1300: minor cleanups | expand

Message

Martin Ågren Jan. 3, 2021, 9:36 a.m. UTC 
There was some discussion around t1300 a while back and how it could
need some major cleanup [1]. This patch series is not that. I'll post an
RFC series on that later. These initial cleanups -- or something like
them -- should be useful regardless of the fate of that future RFC, so
I'm posting them early.

[1] https://lore.kernel.org/git/xmqqlfeuqd8d.fsf@gitster.c.googlers.com/

Martin Ågren (3):
  t1300: remove duplicate test for `--file ../foo`
  t1300: remove duplicate test for `--file no-such-file`
  t1300: don't needlessly work with `core.foo` configs

 t/t1300-config.sh | 72 +++++++++++++++++++++--------------------------
 1 file changed, 32 insertions(+), 40 deletions(-)

Comments

Junio C Hamano Jan. 6, 2021, 6:05 a.m. UTC | #1 
Martin Ågren <martin.agren@gmail.com> writes:

> There was some discussion around t1300 a while back and how it could
> need some major cleanup [1]. This patch series is not that. I'll post an
> RFC series on that later. These initial cleanups -- or something like
> them -- should be useful regardless of the fate of that future RFC, so
> I'm posting them early.
>
> [1] https://lore.kernel.org/git/xmqqlfeuqd8d.fsf@gitster.c.googlers.com/
>
> Martin Ågren (3):
>   t1300: remove duplicate test for `--file ../foo`
>   t1300: remove duplicate test for `--file no-such-file`
>   t1300: don't needlessly work with `core.foo` configs
>
>  t/t1300-config.sh | 72 +++++++++++++++++++++--------------------------
>  1 file changed, 32 insertions(+), 40 deletions(-)

All look sensible.  Thanks.