[Outreachy PATCH v2 0/2] notes-utils: stop using the_repository
- From
- Okhuomon Ajayi <okhuomonajayi54@gmail.com>
- Date
- Nov 3, 2026, 08:49 UTC
- Message-ID
- <20261103084930.15532-1-okhuomonajayi54@gmail.com>
- In-Reply-To
- <20261031104526.46711-1-okhuomonajayi54@gmail.com>
This series removes the_repository from notes-utils.c.
Changes since v1: - Add patch 2, which passes a repository to init_copy_notes_for_rewrite() and drops USE_THE_REPOSITORY_VARIABLE from notes-utils.c, as Patrick suggested. v1 stopped one step short of that.
The callers in builtin/am.c and builtin/notes.c pass the_repository, as those files still use it everywhere. copy() in builtin/notes.c does receive a repo that could be passed down instead, if that is preferred.
Thanks, Patrick, for the review.
Okhuomon Ajayi (2): notes-utils: use the passed repository for notes refs notes-utils: pass the repository to init_copy_notes_for_rewrite()
builtin/am.c | 2 +- builtin/notes.c | 2 +- notes-utils.c | 11 +++++------ notes-utils.h | 3 ++- sequencer.c | 2 +- 5 files changed, 10 insertions(+), 10 deletions(-)
base-commit: 50fe5cc63f9cd6e89b87a2c9ccf96a444f4cfdb7
-- 2.56.0