Skip to content

Fix --resolve-engines in combination with --use-on-cd#1259

Merged
Schniz merged 4 commits intoSchniz:masterfrom
skirsten:fix-resolve-engines-with-use-on-cd
Oct 6, 2024
Merged

Fix --resolve-engines in combination with --use-on-cd#1259
Schniz merged 4 commits intoSchniz:masterfrom
skirsten:fix-resolve-engines-with-use-on-cd

Conversation

@skirsten
Copy link
Contributor

The shell output was not checking for the package.json file.

Fixes #1100

@changeset-bot
Copy link

changeset-bot bot commented Sep 27, 2024

🦋 Changeset detected

Latest commit: 856b5cb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
fnm Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Sep 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fnm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2024 6:05am

@Schniz
Copy link
Owner

Schniz commented Oct 6, 2024

oh damn i didn't see this PR and I committed #1263 with some tests. let me attribute you in that PR

@Schniz
Copy link
Owner

Schniz commented Oct 6, 2024

i love your code more, i'll just add my tests to this PR 😁

@skirsten
Copy link
Contributor Author

skirsten commented Oct 7, 2024

Thanks for merging this.
I had a quick look at your code and noticed that I overlooked the src/shell/windows_cmd/cd.cmd file.

So I believe on Windows with CMD it still missing the fix.

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.

--use-on-cd does not respect --resolve-engines with --version-file-strategy=local

2 participants