[v2,0/3] bundle-uri: copy all bundle references ino the refs/bundle space

Message

Derrick Stolee via GitGitGadget March 1, 2025, 10:33 a.m. UTC

Updating the series by fixing the tests to look in the new space that heads
are placed and added a small test to check for tags.

Scott Chacon (3):
  bundle-uri: copy all bundle references ino the refs/bundle space
  bundle-uri: update bundle clone tests with new refspec path
  bundle-uri: add test for bundle-uri clones with tags

 bundle-uri.c                |   2 +-
 t/t5558-clone-bundle-uri.sh | 203 +++++++++++++++++++++---------------
 2 files changed, 118 insertions(+), 87 deletions(-)


base-commit: cb0ae672aeabefca9704477ea8018ac94f523970
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1897%2Fschacon%2Fsc-more-bundle-refs-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1897/schacon/sc-more-bundle-refs-v2
Pull-Request: https://github.com/git/git/pull/1897

Range-diff vs v1:

 1:  96d78614214 = 1:  b36bc876fe1 bundle-uri: copy all bundle references ino the refs/bundle space
 -:  ----------- > 2:  5e198ba5c66 bundle-uri: update bundle clone tests with new refspec path
 -:  ----------- > 3:  ea204679cb0 bundle-uri: add test for bundle-uri clones with tags

Comments

Derrick Stolee March 3, 2025, 6:49 p.m. UTC | #1

On 3/1/25 5:33 AM, Scott Chacon via GitGitGadget wrote:
> Updating the series by fixing the tests to look in the new space that heads
> are placed and added a small test to check for tags.
> 
> Scott Chacon (3):
>    bundle-uri: copy all bundle references ino the refs/bundle space
>    bundle-uri: update bundle clone tests with new refspec path

Thanks for updating these tests. While I appreciate a small commit, we do
want to make sure that each commit passes the test suite whenever possible,
so these two should be squashed together.

>    bundle-uri: add test for bundle-uri clones with tags

This new test is good to stay isolated into its own commit for easier review.

Thanks,
-Stolee