rogue.js icon indicating copy to clipboard operation
rogue.js copied to clipboard

Warnings about wrong versions

Open v1rtl opened this issue 6 years ago • 2 comments

I ran this to install Rogue:

yarn add @roguejs/cli @roguejs/app react
react-dom react-router react-router-dom

But I already had some deps, such as react, react-dom and express. And during installing I got lots of messages about wrong versions / invalid engines.

My package.json before installation of Rouge:

"dependencies": {
    "express": "^4.16.4",
    "react": "^16.7.0",
    "react-dom": "^16.7.0"
  },
  "scripts": {
    "start": "node server/index.js"
  },
  "devDependencies": {
    "@babel/core": "^7.2.2",
    "@babel/plugin-proposal-class-properties": "^7.2.3",
    "@babel/plugin-transform-function-name": "^7.2.0",
    "@babel/preset-env": "^7.2.3",
    "@babel/preset-react": "^7.0.0",
    "npm-run-all": "^4.1.5"
  }
}

It seems that conflicts shouldn't appear. By the way in my terminal weird things happen:

warning @roguejs/cli > parcel-bundler > htmlnano > cssnano > postcss-merge-rules > [email protected]: Browserslist 2 could fail on
reading Browserslist >3.0 config used in other tools.
warning @roguejs/cli > parcel-bundler > htmlnano > cssnano > postcss-merge-rules > caniuse-api > [email protected]: Browserslist 2
could fail on reading Browserslist >3.0 config used in other tools.
warning @roguejs/app > [email protected]: Please use @loadable/component
warning @roguejs/app > react-native-web > [email protected]: Check out `lodash.merge` or `merge-options` instead.
⠄ @babel/plugin-syntax-nullish-coalescing-op⠁ babel-plugin-syntax-trailing-function-comm⠂ babel-plugin-syntax-trailing-function-comm[2/4] Fetching packages...
warning [email protected]: The engine "gitbook" appears to be invalid.
[3/4] Linking dependencies...
warning "@roguejs/app > [email protected]" has unmet peer dependency "react-art@>=16.5.1".
warning "@roguejs/app > [email protected]" has incorrect peer dependency "[email protected]".

I think some libraries used in Rogue are deprecated and absolete. Nothing crashes, but it isn't cool to see so many warnings during simple installation.

v1rtl avatar Jan 23 '19 13:01 v1rtl

yea just took a look, some dependencies are unused and should be removed - sorry about that.

also, rogue/cli is still early. make sure you're aware of #80 (hopefully parcel v2 will improve things).

alidcast avatar Jan 23 '19 15:01 alidcast

@alidcastano use yarn-check or npm-check CLI tool to automatically look up for obsolete dependencies so such issues won't popup anymore here. Also, why does your app need react-native inside browser? It is used for mobile devices or am I wrong?

Yarn check: https://www.npmjs.com/package/yarn-check

NPM Check: https://www.npmjs.com/package/npm-check

v1rtl avatar Jan 23 '19 20:01 v1rtl