Remove package.json and deno.lock from Deno template

[markmals]

No description provided.

[markmals]

Depends on remix-run/react-router#13744

[johnstonmatt]

this probably solves #132

[markmals]

It will! Just waiting for an official release with the fix in order to open this PR.

[markmals]

@brookslybrand I think this is ready to be merged now

[brookslybrand]

Looks like the CLI also looks for a package.json 🙃

▲  Oh no! The provided template must be a React Router project with a `package.json` file, but that file does not exist in my-react-router-app.

[markmals]

@brookslybrand so, this one is trickier… create-react-router actually reads the package.json to verify that the template contains React Router packages:

    for (let dependency in dependencies) {
      let version = dependencies[dependency];
      if (
        (dependency.startsWith("@react-router/") ||
          dependency === "react-router" ||
          dependency === "react-router-dom") &&
        version === "*"
      ) {
        dependencies[dependency] = semver.prerelease(ctx.reactRouterVersion)
          ? // Templates created from prereleases should pin to a specific version
            ctx.reactRouterVersion
          : "^" + ctx.reactRouterVersion;
      }
    }

Should I add support for checking deno.json(c) as well or remove these checks altogether?

[brookslybrand]

@markmals I think this PR might fix it

[markmals]

Unfortunately, that PR did not fix it.

I can dive deeper and try to fix it. Do you have thoughts on the approach, re: my previous message @brookslybrand? Also, should we go ahead and merge this anyway since the current template fails also and it's not a regression of functionality?

[brookslybrand]

That's annoying/weird. I haven't really looked into it enough to know what needs to be changed to fixed this. Basically whatever in the CLI that is checking for package.json just needs to be expanded I think

The current template works right now because it has a package.json. I'd prefer not to merge this one in because it will make it to where

npx create-react-router@latest --template remix-run/react-router-templates/deno

fails completely