{"thread":{"id":"66701","subject":"[Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","startedAt":"2026-10-29T08:10:27Z","lastAt":"2026-11-02T05:51:01Z","messageCount":5,"participants":["Immaculate Mecha","Okhuomon Ajayi","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"556737","messageId":"CAKUOgLz3sefhypa9yzLszW43uQ=Jor_nRBAJaBXXv0d8M2ePBw@mail.gmail.com","threadId":"66701","inReplyTo":null,"subject":"[Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","fromName":"Immaculate Mecha","fromEmail":"immaculatemecha87@gmail.com","sentAt":"2026-10-29T08:10:27Z","receivedAt":"2026-10-29T08:10:27Z","isPatch":false,"sender":{"key":"immaculatemecha87@gmail.com","avatar":null},"body":"Hi,\n\nI'm preparing a microproject contribution for Outreachy and have been\nexploring a small test cleanup in Git.\n\nIn t/t0081-find-pack.sh, line 35 currently uses:\n\ntest -f \"$head_commit_pack\" &&\n\nI'm considering replacing it with Git's test helper:\n\ntest_path_is_file \"$head_commit_pack\" &&\n\nI checked the file's history using git blame and confirmed that this\nis the only test -f, test -e, or test -d check in that script.\n\nWould this be an appropriate and useful microproject, or is there a\nreason to leave this check as it is?\n\nI have cloned Git, built it with NO_RUST=1, and run the test suite. I\nhave not made any changes to the source yet.\n\nThank you for your time.\n\n"},{"id":"556744","messageId":"CAFpMFfA17dnuLSRt97BrM0u3cHAifJybr_Zc2ahr_zq2nspgsw@mail.gmail.com","threadId":"66701","inReplyTo":"CAKUOgLz3sefhypa9yzLszW43uQ=Jor_nRBAJaBXXv0d8M2ePBw@mail.gmail.com","subject":"Re: [Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","fromName":"Okhuomon Ajayi","fromEmail":"okhuomonajayi54@gmail.com","sentAt":"2026-10-29T09:03:16Z","receivedAt":"2026-10-29T09:03:16Z","isPatch":false,"sender":{"key":"okhuomonajayi54@gmail.com","avatar":null},"body":"On Thu, Oct 29, 2026 at 9:10 AM Immaculate Mecha\n wrote:\n>\n> Hi,\n>\n> I'm preparing a microproject contribution for Outreachy and have been\n> exploring a small test cleanup in Git.\n>\n> In t/t0081-find-pack.sh, line 35 currently uses:\n>\n> test -f \"$head_commit_pack\" &&\n>\n> I'm considering replacing it with Git's test helper:\n>\n> test_path_is_file \"$head_commit_pack\" &&\n>\n> I checked the file's history using git blame and confirmed that this\n> is the only test -f, test -e, or test -d check in that script.\n>\n> Would this be an appropriate and useful microproject, or is there a\n> reason to leave this check as it is?\n\n\nHi Immaculate,\n\nWelcome! I think Shashank S already changed this exact line in t0081\n\nhttps://lore.kernel.org/git/pull.2254.v2.git.1792061236183.gitgitgadget@gmail.com/\n\nBefore choosing a microproject, it helps to search the list archive\nat https://lore.kernel.org/git/ for the file name, and to check the\n\"What's cooking\" reports, to make sure nobody is already working on\nit.\n\nGood luck!\nOkhuomon\n\n"},{"id":"556745","messageId":"CAKUOgLy_jS-x5jZcQZmjHzLV=TnPi56iQ9v00hUXmX9Kudorjg@mail.gmail.com","threadId":"66701","inReplyTo":"CAFpMFfA17dnuLSRt97BrM0u3cHAifJybr_Zc2ahr_zq2nspgsw@mail.gmail.com","subject":"Re: [Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","fromName":"Immaculate Mecha","fromEmail":"immaculatemecha87@gmail.com","sentAt":"2026-10-29T09:54:18Z","receivedAt":"2026-10-29T09:54:18Z","isPatch":false,"sender":{"key":"immaculatemecha87@gmail.com","avatar":null},"body":"Hi,\n\nThank you for the welcome and for pointing me to the existing work.\nI'll review the mailing-list archive and the What's cooking reports\nmore carefully before choosing another microproject.\n\nBest,\nImmaculate\n\n\nOn Thu, Oct 29, 2026 at 12:03 PM Okhuomon Ajayi\n wrote:\n>\n> On Thu, Oct 29, 2026 at 9:10 AM Immaculate Mecha\n> wrote:\n> >\n> > Hi,\n> >\n> > I'm preparing a microproject contribution for Outreachy and have been\n> > exploring a small test cleanup in Git.\n> >\n> > In t/t0081-find-pack.sh, line 35 currently uses:\n> >\n> > test -f \"$head_commit_pack\" &&\n> >\n> > I'm considering replacing it with Git's test helper:\n> >\n> > test_path_is_file \"$head_commit_pack\" &&\n> >\n> > I checked the file's history using git blame and confirmed that this\n> > is the only test -f, test -e, or test -d check in that script.\n> >\n> > Would this be an appropriate and useful microproject, or is there a\n> > reason to leave this check as it is?\n>\n>\n> Hi Immaculate,\n>\n> Welcome! I think Shashank S already changed this exact line in t0081\n>\n> https://lore.kernel.org/git/pull.2254.v2.git.1792061236183.gitgitgadget@gmail.com/\n>\n> Before choosing a microproject, it helps to search the list archive\n> at https://lore.kernel.org/git/ for the file name, and to check the\n> \"What's cooking\" reports, to make sure nobody is already working on\n> it.\n>\n> Good luck!\n> Okhuomon\n\n"},{"id":"556812","messageId":"xmqq7bj02wak.fsf@gitster.g","threadId":"66701","inReplyTo":"CAFpMFfA17dnuLSRt97BrM0u3cHAifJybr_Zc2ahr_zq2nspgsw@mail.gmail.com","subject":"Re: [Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-10-29T18:31:31Z","receivedAt":"2026-10-29T18:31:31Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Okhuomon Ajayi writes:\n\n> Before choosing a microproject, it helps to search the list archive\n> at https://lore.kernel.org/git/ for the file name, and to check the\n> \"What's cooking\" reports, to make sure nobody is already working on\n> it.\n\nGreat advice. Reading others' patches and reviews they have\nreceived will prepare a newcomer to avoid making the same mistakes\nas they already have made. Reading Documentation/SubmittingPatches\nwould also help, but that goes without saying ;-)\n\n"},{"id":"557115","messageId":"CAKUOgLy8iSh8RtSNoTp6KtLhe0SkU04DcTstERdbnN7xSDb9bQ@mail.gmail.com","threadId":"66701","inReplyTo":"xmqq7bj02wak.fsf@gitster.g","subject":"Re: [Outreachy] Microproject idea: modernize a file-existence check in t/t0081-find-pack.sh","fromName":"Immaculate Mecha","fromEmail":"immaculatemecha87@gmail.com","sentAt":"2026-11-02T05:51:01Z","receivedAt":"2026-11-02T05:51:01Z","isPatch":false,"sender":{"key":"immaculatemecha87@gmail.com","avatar":null},"body":"Hello, i sent this project request \"[Outreachy] GIT_DIR inherited by\nshell aliases in linked worktrees\" , and i haven't gotten feedback yet\non whether i should proceed or not. I will appreciate your feedback.\n\n\nOn Thu, Oct 29, 2026 at 9:31 PM Junio C Hamano wrote:\n>\n> Okhuomon Ajayi writes:\n>\n> > Before choosing a microproject, it helps to search the list archive\n> > at https://lore.kernel.org/git/ for the file name, and to check the\n> > \"What's cooking\" reports, to make sure nobody is already working on\n> > it.\n>\n> Great advice. Reading others' patches and reviews they have\n> received will prepare a newcomer to avoid making the same mistakes\n> as they already have made. Reading Documentation/SubmittingPatches\n> would also help, but that goes without saying ;-)\n\n"}]}