Skip to content

Commit

Permalink
Don't pass --node-modules-dir flag to deno install
Browse files Browse the repository at this point in the history
  • Loading branch information
redabacha committed Sep 2, 2024
1 parent 1e48a52 commit b94e223
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 11 deletions.
6 changes: 1 addition & 5 deletions packages/create-remix/__tests__/create-remix-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -926,11 +926,7 @@ describe("create-remix CLI", () => {

expect(execa).toHaveBeenCalledWith(
"deno",
expect.arrayContaining([
"install",
"--node-modules-dir=true",
"--no-lock",
]),
expect.arrayContaining(["install", "--no-lock"]),
expect.objectContaining({ env: { DENO_FUTURE: "1" } })
);
delete process.versions.deno;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{
"nodeModulesDir": true,
"imports": {
"is-odd": "npm:[email protected]"
}
Expand Down
7 changes: 1 addition & 6 deletions packages/create-remix/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -672,12 +672,7 @@ async function installDependencies({
try {
await execa(
pkgManager,
[
"install",
...(pkgManager === "deno"
? ["--node-modules-dir=true", "--no-lock"]
: []),
],
["install", ...(pkgManager === "deno" ? ["--no-lock"] : [])],
{
cwd,
env: pkgManager === "deno" ? { DENO_FUTURE: "1" } : undefined,
Expand Down

0 comments on commit b94e223

Please sign in to comment.