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

[TECH] Monter de version caniuse-lite #3473

Merged
merged 21 commits into from
Sep 10, 2021

Conversation

Anne-Gaelle-S
Copy link
Contributor

@Anne-Gaelle-S Anne-Gaelle-S commented Sep 9, 2021

🦄 Problème

Quand la dépendance caniuse-lite détecte qu'elle n'est pas à jour, elle devient très verbeuse au lancement des applications front (mais pas pour tout le monde apparament) :
image

Les explications de pourquoi on doit mettre cette dépendance à jour régulièrement sont ici : https://github.com/browserslist/browserslist#browsers-data-updating.
Mais en gros, c'est pour que caniuse puisse déterminer la véritable "dernière version" des navigateurs. Et aussi pour éviter les conflits lorsque plusieurs versions de caniuse sont utilisées dans les dépendances.

🤖 Solution

Mettre à jour caniuse.

💯 Pour tester

  • vérifier la ci
  • lancer une des applications front
  • constater que caniuse se taise 😌
    Grosso modo le build d'une app front doit se résumer à :

image

@pix-service
Copy link
Contributor

@jbuget
Copy link
Contributor

jbuget commented Sep 9, 2021

🤔 Je ne vois que des modifs de fichiers package-lock.json. Ca veut dire que si je fais un npm install sur mon poste (et pas npm ci, je risque de générer un fichier lock file différent. Par défaut c'est chiant, lors des commits. Au pire, si je ne fais pas attention, je risque de casser le fix. Je ne suis pas certain que ça corrige parfaitement le problème.

@sbedeau
Copy link
Contributor

sbedeau commented Sep 9, 2021

🤔 Je ne vois que des modifs de fichiers package-lock.json. Ca veut dire que si je fais un npm install sur mon poste (et pas npm ci, je risque de générer un fichier lock file différent. Par défaut c'est chiant, lors des commits. Au pire, si je ne fais pas attention, je risque de casser le fix. Je ne suis pas certain que ça corrige parfaitement le problème.

Oui c'est juste. Ça vient notamment du fait que caniuse-lite est une dépendance de nos dépendances.
Ça veut dire que l'on monter les paquets associés pour que ces warnings nous laissent tranquilles :/

Résultat de npm ls caniuse-lite dans ./mon-pix

[email protected]
├─┬ @sentry/[email protected]
│ └─┬ @embroider/[email protected]
│   └─┬ @babel/[email protected]
│     └─┬ @babel/[email protected]
│       └─┬ [email protected]
│         └── [email protected] 
├─┬ [email protected]
│ └─┬ @babel/[email protected]
│   └─┬ [email protected]
│     └─┬ [email protected]
│       └── [email protected]  deduped
├─┬ [email protected]
│ └─┬ @babel/[email protected]
│   └─┬ @babel/[email protected]
│     └─┬ [email protected]
│       └── [email protected] 
├─┬ [email protected]
│ ├─┬ @babel/[email protected]
│ │ └─┬ @babel/[email protected]
│ │   └─┬ [email protected]
│ │     └── [email protected] 
│ ├─┬ @babel/[email protected]
│ │ └─┬ [email protected]
│ │   └── [email protected]  deduped
│ └─┬ @babel/[email protected]
│   ├─┬ [email protected]
│   │ └─┬ @babel/[email protected]
│   │   └─┬ @babel/[email protected]
│   │     └─┬ [email protected]
│   │       └── [email protected] 
│   └─┬ [email protected]
│     └─┬ [email protected]
│       └─┬ [email protected]
│         └── [email protected] 
├─┬ [email protected]
│ └─┬ [email protected]
│   └─┬ [email protected]
│     └── [email protected]  deduped
├─┬ [email protected]
│ └─┬ [email protected]
│   └─┬ @babel/[email protected]
│     └─┬ [email protected]
│       └── [email protected]  deduped
├─┬ [email protected]
│ └─┬ [email protected]
│   ├─┬ [email protected]
│   │ └── [email protected] 
│   └── [email protected] 
├─┬ [email protected]
│ └─┬ [email protected]
│   └─┬ @babel/[email protected]
│     └─┬ [email protected]
│       └── [email protected]  deduped
└─┬ [email protected]
  ├─┬ @stylelint/[email protected]
  │ └─┬ @babel/[email protected]
  │   └─┬ @babel/[email protected]
  │     └─┬ [email protected]
  │       └── [email protected] 
  └─┬ [email protected]
    ├─┬ [email protected]
    │ └── [email protected]  deduped
    └── [email protected]  deduped

@Anne-Gaelle-S
Copy link
Contributor Author

npm ls caniuse-lite

Je change ça !

@sbedeau
Copy link
Contributor

sbedeau commented Sep 9, 2021

npm ls caniuse-lite

Je change ça !

Il y a la PR #3382 qui traite indirectement et partiellement le sujet déjà. Mais ça bloque pas mal

@Anne-Gaelle-S Anne-Gaelle-S force-pushed the tech-fix-caniuse-lite-warnings branch 2 times, most recently from c5f6542 to 2b32a7d Compare September 10, 2021 08:20
Copy link
Contributor

@sbedeau sbedeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

louange: Merci pour tout ce travail et la démarche "c'est douloureux donc on soulage ASAP" 🙏

@Anne-Gaelle-S Anne-Gaelle-S force-pushed the tech-fix-caniuse-lite-warnings branch from 2b32a7d to d7296e9 Compare September 10, 2021 09:41
Copy link
Member

@laura-bergoens laura-bergoens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow tu rigoles plus avec les PRs là 😅

@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-fix-caniuse-lite-warnings branch from d7296e9 to 1ab1334 Compare September 10, 2021 13:10
@pix-service-auto-merge pix-service-auto-merge deleted the tech-fix-caniuse-lite-warnings branch September 10, 2021 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants