t1400: modernize style
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 patches 10 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | t1400: rename test descriptions to be uniquet/ | +4 | 1 reply | |
| 2/5 | t1400: set core.logAllRefUpdates in "logged by touch" testst/ | +12 | 2 replies | |
| 3/5 | t1400: use test_path_is_* helperst/ | +22 | 1 reply | |
| 4/5 | t1400: remove a set of unused output filest/ | +2 | 1 reply | |
| 5/5 | t1400: use test_when_finished for cleanupt/ | +41 | 1 reply |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/5] t1400: modernize style
These patches follow up on Peff's suggestion to modernize the style in t1400-update-ref.sh.
[email protected]
The first two commits aren't concerned with "modernizing" the tests, but instead address issues that I noticed while looking more closely at t1400.
I also considered
* making the quoting/spacing/breaks around the test descriptions and bodies more consistent, but I think this leads to too much code churn.
* moving the here-documents for log creation into the following tests, but I don't think it's worth it because it makes already long lines even longer.
t1400: rename test descriptions to be unique
t1400: set core.logAllRefUpdates in "logged by touch" tests
t1400: use test_path_is_* helpers
t1400: remove a set of unused output files
t1400: use test_when_finished for cleanup
t/t1400-update-ref.sh | 154 +++++++++++++++++++++++++------------------------- 1 file changed, 78 insertions(+), 76 deletions(-)
Signature · 1 line
2.12.0