Skip to content

Produce sparse_paths_type option for sparse checkouts to designate "cone" or "no-cone" - #475

Draft
paulhovey wants to merge 1 commit into
jacebrowning:mainfrom
paulhovey:sparse-cones
Draft

Produce sparse_paths_type option for sparse checkouts to designate "cone" or "no-cone"#475
paulhovey wants to merge 1 commit into
jacebrowning:mainfrom
paulhovey:sparse-cones

Conversation

@paulhovey

Copy link
Copy Markdown

I believe this should resolve #360 and #363 . It exposes a sparse_paths_type which will pass --cone or --no-cone` to git.

It also updates the regex package, as mentioned in #363 .

I'm open to any feedback on this, I'm just someone with the same issue and access to GLM-5.2 🙂

@jacebrowning
jacebrowning self-requested a review June 27, 2026 22:27
@jacebrowning

Copy link
Copy Markdown
Owner

@paulhovey thanks for the PR! I'd really like to see a solution here doesn't require a new field for such a specific usage. Can the need for "cone" be inferred at all? Would always passing --no-cone work for the majority of cases? Please discuss further on the issues if needed.

I'll pull the regex upgrade into another PR.

@jacebrowning
jacebrowning marked this pull request as draft July 23, 2026 23:24
@jacebrowning
jacebrowning removed their request for review July 23, 2026 23:24
@paulhovey

Copy link
Copy Markdown
Author

Hi Jace, understood about not wanting to introduce a new config option, but I'm not sure a better way off the top of my head. Using --no-cone is not recommended, so I think staying with the default is what gitman should be doing: https://git-scm.com/docs/git-sparse-checkout#_internalsnon_cone_problems

However, some people may have a specific reason for needing --no-cone, and this would enable them. I'll think on this a little more, but an extra toggle might be the only way to do this, since I don't think it can be strictly inferred.

Thanks for pulling the regex into a separate PR!

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.

Add --no-cone option to sparse checkout

2 participants