On 4/14/2022 2:57 PM, Josh Steadmon wrote: > On 2022.04.07 16:37, Derrick Stolee via GitGitGadget wrote: >> From: Derrick Stolee <derrickstolee@xxxxxxxxxx> >> This changes the behavior of 'git show :<sparse-dir>', but does not >> bring it entirely into alignment with a full index case. It specifically >> hits the wrong error message within diagnose_invalid_index_path(). That >> error message will be corrected in a future change. ... >> + test_must_fail git -C sparse-index show :folder1/ 2>err && >> + grep "is in the index, but not at stage 0" err >> ' > > It might be worth a note that we're demonstrating the current behavior > here, but this is not the desired end-state. In other words, explicitly > note that this is the "wrong error message" referred to in the commit > message. This is mentioned in the commit message. Since the line will be changed in the next patch, I didn't think a test comment was worth it. Thanks, -Stolee
