Skip to content

Conversation

@mjcheetham
Copy link
Member

Add the --ref-format option to the scalar clone command. This will allow users to opt-in to creating a Scalar repository using alternative ref storage backends, such as reftable. Example:

scalar clone --ref-format reftable $URL

Add ability to run Git commands for Scalar by passing a struct strvec
rather than having to use varargs.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
@mjcheetham mjcheetham requested a review from dscho December 17, 2025 14:12
dscho
dscho previously approved these changes Dec 17, 2025
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will also need a corresponding PR for vfs-2.52.0, unfortunately.

derrickstolee
derrickstolee previously approved these changes Dec 17, 2025
Add the `--ref-format` option to the `scalar clone` command. This will
allow users to opt-in to creating a Scalar repository using alternative
ref storage backends, such as reftable. Example:

  scalar clone --ref-format reftable $URL

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
@mjcheetham
Copy link
Member Author

We will also need a corresponding PR for vfs-2.52.0, unfortunately.

#832

@mjcheetham
Copy link
Member Author

We will also need a corresponding PR for vfs-2.52.0, unfortunately.

Actually.. I guess that we'll just want the 2.52 targeting PR, right? I only target 2.51.2 in this PR since it's the default branch still.

@dscho
Copy link
Member

dscho commented Dec 18, 2025

we'll just want the 2.52 targeting PR, right?

That would at least allow us to use git repo in the tests...

@derrickstolee
Copy link

@mjcheetham: This is something that we could send upstream, too, right?

@dscho
Copy link
Member

dscho commented Dec 18, 2025

This is something that we could send upstream, too, right?

Yes!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants