agilgur5 / agilgur5/front-end-base

use NpmInstallPlugin to fix unintuitive-ness with compilation after installing a module

Abierto
#3 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
0
Forks
1
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

The basic problem is explained well in CRA: https://github.com/facebook/create-react-app/issues/186 . Unintuitive but does work eventually (I don't think I even knew it would work on the second save though, pretty sure I just thought it didn't work period :o).

https://github.com/facebook/create-react-app/pull/212#issuecomment-235284840 and the sentiment of this issue in general is similar to some of my own reservations -- I'd much rather `npm install` the package myself and the auto install is different from the usual process of development without running a watcher or this plugin.
In this case, I think using [react-dev-utils](https://www.npmjs.com/package/react-dev-utils)'s [WatchMissingNodeModulesPlugin](https://www.npmjs.com/package/react-dev-utils#new-watchmissingnodemodulespluginnodemodulespath-string) might make more sense to include. That might be the only thing I would use from that package though (so installing the whole thing is a bit wasteful, but ¯\\\_(ツ)_/¯ )

Another important note about the NpmInstallPlugin is that it currently doesn't work with Webpack 4 and has a number of outstanding issues: https://github.com/webpack-contrib/npm-install-webpack-plugin/pull/123 . https://www.npmjs.com/package/webpack-plugin-install-deps is a fork from the PR that supports Webpack 4 fwiw.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.