[GitHub] [cordova-cli] QuentinFarizon edited a comment on issue #303: Please restore `--nofetch` flag to support Yarn

[GitBox <gi...@apache.org>]

QuentinFarizon edited a comment on issue #303:
URL: https://github.com/apache/cordova-cli/issues/303#issuecomment-643806049


   @vzts @raphinesse I can confirm that npm install is not called when pre-yarn installed environment
   
   **BUT** npm install is called when a cordova plugin has been added **using `yarn add` with a git repository url** instead of the registry name (screwing node_modules as describes in this issue).
   Ex : `yarn add git+https://github.com/videmort/cordova-plugin-referrer.git`
   Vs standard `yarn add cordova-plugin-referrer`
   
   This is a normal way to install dependencies, and useful when installing forks of cordova plugins. It make it impossible to have yarn + plugins referrenced by git urls
   
   Please either re-add --nofetch, or make it use yarn if configured as such, or just check by looking in node_modules/cordova-plugin-referrer instead of a more complicated way (which fails here)


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org