Skip to content

fix: issue-6960 Correctly handle scenario where prefix is the CWD #8269

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from

Conversation

owlstronaut
Copy link
Contributor

@owlstronaut owlstronaut commented May 2, 2025

closes #6960

related #7208

resolves feedback from #7208 (comment) and #7208 (review)

Manually verified the fix on windows

@owlstronaut owlstronaut force-pushed the 6960_fix_local_directory_prefix_usage branch from bc7a7b5 to 48afa7b Compare May 2, 2025 21:02
@owlstronaut owlstronaut marked this pull request as ready for review May 2, 2025 21:03
@owlstronaut owlstronaut requested a review from a team as a code owner May 2, 2025 21:03
@owlstronaut owlstronaut force-pushed the 6960_fix_local_directory_prefix_usage branch from 48afa7b to ef0f993 Compare May 2, 2025 21:05
@owlstronaut owlstronaut changed the title 6960 fix local directory prefix usage fix: issue-6960 Correctly handle scenario where prefix is the CWD May 2, 2025
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.

[BUG] npm install --prefix="$PWD" adds invalid dependency to package.json
2 participants