Closed iqre8 closed 5 months ago
@iqre8 read contributing file, we're using pnpm as package manager
Upps, I didn't read full issue...
@iqre8 read contributing file, we're using pnpm as package manager
I ran into this with npm v10.1.0
as well. The docs on https://vite-pwa-org.netlify.app/examples/ have conflicting information. It says both,
If you don't have installed PNPM, you must install it globally via npm: npm install -g pnpm
But then further down says,
We use PNPM but should work with any package manager, for example, with YARN: yarn && yarn build
Yarn install gracefully fails on the workspace:*
package.json entry, but npm
fails.
If the requirement to run the examples requires pnpm
, then I think it would be helpful to update the language referenced above
If you want to use anther package manager you should update the configuration in your local properly, we use pnpm workspaces.
Can I raise the vue-router example uses nr
in many of the package.json scripts. nr
is not in the devDependencies and that is the recommendation from the nr docs https://github.com/darkobits/nr however when I install nr in my test environment the example project still won't start as it is looking for an nr
config file which is not present in the example repo given. I'm just thoroughly confused by this I must be doing something wrong, can you shed any light on my questions?
@michaeltoohig nr
is a script from ni
dependency: https://github.com/vite-pwa/vite-plugin-pwa/blob/main/package.json#L115
EDIT: check the contributing guide, there is a link to previous repo IIRC.
Expected behaviour: That we could install the dependancies and run the example. Actual behaviour: When doing an npm install, it could not resolve the dependancies. See below for errors Changed in package.json from "vite-plugin-pwa": "workspace:*", to "vite-plugin-pwa": "^0.14.0",
After doing npm run dev -- --host I however still see 'No matching service worker detected' in chrome -> developer tools ->Application -> Manifest