Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: feat: use Yarn Plug'n'Play #32

Closed
wants to merge 10 commits into from
Closed

WIP: feat: use Yarn Plug'n'Play #32

wants to merge 10 commits into from

Conversation

Kocal
Copy link
Contributor

@Kocal Kocal commented Oct 6, 2018

Liens relatifs :

En résumé, Yarn Plug'n'Play est une nouvelle façon de résoudre les dépendances Node.js avec Yarn 1.11 (qui n'est pas encore officiellement release 😞 ).
Avec Yarn PnP, on n'aura plus de dossier node_modules dans le projet, ça ira directement piocher les dépendances dans le dossier de cache de Yarn.

Par exemple si on a plusieurs projets qui utilisent la même dépendances (par exemple la version X.Y d'ESLint), on aura qu'une dépendance d'ESLint X.Y sur le disque, elle ne sera pas dupliquée et n'ira pas alourdir l'espace disque pour rien.

Ça offre un gros gain sur Yprox/ALP/Enigma/les projets avec le deployment helper d'Ansible, et notamment sur le Yprox staging.

Par exemple sur Yprox, on a 420 Mo de dépendances Node.js sur admin et 370 Mo sur sites.
Si on utilise Yarn PnP, déjà on économisera pas mal de Mo car on a certainement un tas de dépendances en commun, mais on en économisera encore plus entre les différentes releases Ansible. Et ceci se confirmera encore plus sur Ziggs (~ 60 instances du projet Yprox).


Sinon, cette PR avait pour but de setup le plugin rollup de compatibilité PnP x)

@Kocal Kocal force-pushed the feat/setup-yarn-pnp branch from 99c6def to 7e4b4f4 Compare October 6, 2018 06:09
@Kocal Kocal changed the title feat: use Yarn Plug'n'Play WIP: feat: use Yarn Plug'n'Play Oct 6, 2018
@Kocal Kocal force-pushed the feat/setup-yarn-pnp branch from 9f80765 to 35496cb Compare October 6, 2018 08:33
@Kocal Kocal force-pushed the feat/setup-yarn-pnp branch from 5c7deaa to 69d1a89 Compare October 7, 2018 06:04
@Kocal Kocal closed this Oct 7, 2018
@Kocal
Copy link
Contributor Author

Kocal commented Oct 7, 2018

Ouais bon, c'était peut-être trop tôt pour setup PnP sur ce projet, entre :

  • certaines dépendances qui ne sont pas résolvables
  • le fait qu'on ne puisse pas appeler deux .pnp.js (pour test-app et le cli)
  • ESLint

bah on n'est pas sortis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant