Skip to content

fix: pass _isRoot context where missing#9206

Merged
wraithgar merged 1 commit intolatestfrom
gar/allow-git-fix
Apr 15, 2026
Merged

fix: pass _isRoot context where missing#9206
wraithgar merged 1 commit intolatestfrom
gar/allow-git-fix

Conversation

@wraithgar
Copy link
Copy Markdown
Member

fixes: #9189

@wraithgar wraithgar requested a review from a team as a code owner April 8, 2026 17:32
@wraithgar wraithgar force-pushed the gar/allow-git-fix branch from 6bab875 to 23d9832 Compare April 8, 2026 17:34
Copy link
Copy Markdown
Contributor

@owlstronaut owlstronaut left a comment

Choose a reason for hiding this comment

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

LGTM

@akhenry
Copy link
Copy Markdown

akhenry commented Apr 8, 2026

@wraithgar
Copy link
Copy Markdown
Member Author

The test was fine, it's a negative test (i.e. refuses a situation). There is no positive test (i.e. allows the situation) and that's what would have caught this.

@akhenry
Copy link
Copy Markdown

akhenry commented Apr 8, 2026

The test was fine, it's a negative test (i.e. refuses a situation). There is no positive test (i.e. allows the situation) and that's what would have caught this.

Even if it's a negative test, it doesn't actually set allow-git to root, so it doesn't seem to be testing what it says it's testing:

Screenshot 2026-04-08 at 1 58 29 PM

@akhenry
Copy link
Copy Markdown

akhenry commented Apr 8, 2026

In any case, thank you for addressing this issue so quickly! Appreciate it.

@owlstronaut owlstronaut reopened this Apr 15, 2026
@wraithgar wraithgar merged commit dcad8ec into latest Apr 15, 2026
32 of 35 checks passed
@wraithgar wraithgar deleted the gar/allow-git-fix branch April 15, 2026 17:54
github-actions bot pushed a commit that referenced this pull request Apr 15, 2026
@github-actions

This comment was marked as outdated.

wraithgar pushed a commit that referenced this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] allow-git=root does not allow root level git dependencies to be installed

3 participants