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

[PATCHv4 0/9] Handling empty notes

From
Johan Herland <johan@herland.net>
Date
Nov 9, 2014, 12:30 UTC
Message-ID
<1415536255-19961-1-git-send-email-johan@herland.net>
Changes v3 -> v4:
 - Incorporate more feedback from Junio:
    - Patch #1: s/git config --unset/test_unconfig/
 - Add patches #3 - #6 to refactor add()/append_edit()/create_note()
   for supporting the direction suggested by Junio
 - Revamp the last patch to more fully modernize t3301
Have fun! :)
...Johan
Johan Herland (9):
  builtin/notes: Fix premature failure when trying to add the empty blob
  t3301: Verify that 'git notes' removes empty notes by default
  builtin/notes: Improve naming
  builtin/notes: Refactor note file path into struct note_data
  builtin/notes: Simplify early exit code in add()
  builtin/notes: Split create_note() to clarify add vs. remove logic
  builtin/notes: Add --allow-empty, to allow storing empty notes
  notes: Empty notes should be shown by 'git log'
  t3301: Modernize
 Documentation/git-notes.txt |   12 +-
 builtin/notes.c             |  235 +++++----
 notes.c                     |    3 +-
 t/t3301-notes.sh            | 1189 ++++++++++++++++++++-----------------------
 4 files changed, 698 insertions(+), 741 deletions(-)
-- 
2.1.1.392.g062cc5d
Next: Johan Herland
Message 1 of 17 in “[PATCHv4 0/9] Handling empty notes”
  1. Johan HerlandNov 9, 2014
  2. 1/9 builtin/notes: Fix premature failure when trying to add the empty blobJohan Herland, Nov 9, 2014
  3. 2/9 t3301: Verify that 'git notes' removes empty notes by defaultJohan Herland, Nov 9, 2014
  4. 3/9 builtin/notes: Improve namingJohan Herland, Nov 9, 2014
  5. 4/9 builtin/notes: Refactor note file path into struct note_dataJohan Herland, Nov 9, 2014
  6. 5/9 builtin/notes: Simplify early exit code in add()Johan Herland, Nov 9, 2014
  7. Junio C HamanoNov 10, 2014
  8. Johan HerlandNov 11, 2014
  9. Junio C HamanoNov 11, 2014
  10. 6/9 builtin/notes: Split create_note() to clarify add vs. remove logicJohan Herland, Nov 9, 2014
  11. 7/9 builtin/notes: Add --allow-empty, to allow storing empty notesJohan Herland, Nov 9, 2014
  12. 8/9 notes: Empty notes should be shown by 'git log'Johan Herland, Nov 9, 2014
  13. 9/9 t3301: ModernizeJohan Herland, Nov 9, 2014
  14. Junio C HamanoNov 10, 2014
  15. Johan HerlandNov 11, 2014
  16. Eric SunshineNov 11, 2014
  17. Johan HerlandNov 11, 2014

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.