erisu edited a comment on pull request #874:
URL: https://github.com/apache/cordova-lib/pull/874#issuecomment-916703741


   Can you explain why you have the platform package defined in both 
`dependencies` and `devDependencies`?
   
   Having it defined in both is not considered valid with npm...
   
   For example, if you run the following commands
   
   ```shell
   mkdir testProject
   cd testProject
   npm init -y
   npm i cordova-android
   npm i -D cordova-android
   ```
   
   You will see that there is only one entry for `cordova-android`. The npm 
process would just move the package from the `dependencies` list to 
`devDependencies`.
   
   That is why cordova-lib is using an `if else` statement. It checks for one 
and if it doesn't exist then it should be in the other.
   
   I think the solution is to delete one of them, and preferably keep it in 
`devDependencies` as the package is not something bundled with the app.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to