git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/7] Modernize t9300-fast-import

From
Johannes Sixt <j6t@kdbg.org>
Date
Nov 19, 2015, 19:09 UTC
Message-ID
<cover.1447959452.git.j6t@kdbg.org>

Some time ago, I had to dig into t9300-fast-import and found it quite unhelpful that it does not follow our modern best-practices. This series brings it up-to-date. I thought I submit it now while it is quiet in the area.

The larger patches are best viewed using -w -color-words because the regular patch text is ... overwhelming.

Improving shell coding style is outside the scope of this series. I mean fixing eyesores such as 'cat >foo <<EOF && cat foo | sort > bar', or minor things such as quoting <<\EOF when the here-doc does not require substitutions.

In case the large patches don't make it to the list, the series is also available from

  https://github.com/j6t/git.git modernize-t9300
Johannes Sixt (7):
  modernize t9300: single-quote placement and indentation
  modernize t9300: use test_must_fail
  modernize t9300: use test_must_be_empty
  modernize t9300: wrap lines after &&
  modernize t9300: use test_when_finished for clean-up
  modernize t9300: mark here-doc words to ignore tab indentation
  modernize t9300: move test preparations into test_expect_success
 t/t9300-fast-import.sh | 3629 ++++++++++++++++++++++++------------------------
 1 file changed, 1822 insertions(+), 1807 deletions(-)
-- 
2.6.2.337.ga235d84
Next: Johannes Sixt
Message 1 of 10 in “Modernize t9300-fast-import”
  1. 0/7 Modernize t9300-fast-importJohannes Sixt, Nov 19, 2015
  2. 1/7 modernize t9300: single-quote placement and indentationJohannes Sixt, Nov 19, 2015
  3. 2/7 modernize t9300: use test_must_failJohannes Sixt, Nov 19, 2015
  4. 3/7 modernize t9300: use test_must_be_emptyJohannes Sixt, Nov 19, 2015
  5. 4/7 modernize t9300: wrap lines after &&Johannes Sixt, Nov 19, 2015
  6. 5/7 modernize t9300: use test_when_finished for clean-upJohannes Sixt, Nov 19, 2015
  7. 6/7 modernize t9300: mark here-doc words to ignore tab indentationJohannes Sixt, Nov 19, 2015
  8. 7/7 modernize t9300: move test preparations into test_expect_successJohannes Sixt, Nov 19, 2015
  9. Eric SunshineNov 20, 2015
  10. Jeff KingNov 20, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.