On Mon, Oct 21, 2024 at 09:32:20PM +0800, shejialuo wrote: > shejialuo (9): > ref: initialize "fsck_ref_report" with zero > ref: check the full refname instead of basename > ref: initialize target name outside of check functions > ref: support multiple worktrees check for refs > ref: port git-fsck(1) regular refs check for files backend > ref: add more strict checks for regular refs > ref: add basic symref content check for files backend > ref: check whether the target of the symref is a ref > ref: add symlink ref content check for files backend > > Documentation/fsck-msgids.txt | 35 +++ > builtin/refs.c | 12 +- > fsck.h | 6 + > refs.c | 7 +- > refs.h | 3 +- > refs/debug.c | 5 +- > refs/files-backend.c | 187 ++++++++++++-- > refs/packed-backend.c | 8 +- > refs/refs-internal.h | 5 +- > refs/reftable-backend.c | 3 +- > t/t0602-reffiles-fsck.sh | 457 +++++++++++++++++++++++++++++++++- > 11 files changed, 693 insertions(+), 35 deletions(-) Great, thanks for the new round. Looking at the inter-diff, it looks like this round also needs a fresh review. I'm catching up on new threads from the weekend, so I'll put this on my review queue. But in the meantime, if your mentors can look at it, that would be much appreciated. Thanks, Taylor
