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] Montée de version des packages eslint, nuxt et autoprefixer #233

Merged
merged 3 commits into from
Dec 8, 2020

Conversation

celineung
Copy link
Contributor

🦄 Problème

Certains packages ne sont plus à jour:

Screenshot 2020-12-07 at 16 44 35

🤖 Solution

Mettre à jour les packages

💯 Pour tester

Se rendre sur les applications et vérifier la non-régression

@celineung celineung added the cross-team Toutes les équipes de dev label Dec 7, 2020
@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@sbedeau
Copy link
Contributor

sbedeau commented Dec 7, 2020

Est-ce que ces paquets sont utiles à l'endroit où ils se trouvent ?

@celineung
Copy link
Contributor Author

Est-ce que ces paquets sont utiles à l'endroit où ils se trouvent ?

C'est-à-dire: est-ce que ces packages sont utilisés? Parce qu'il s'agit de nuxt et du linter, donc ils sont utiles...

@sbedeau
Copy link
Contributor

sbedeau commented Dec 7, 2020

Est-ce que ces paquets sont utiles à l'endroit où ils se trouvent ?

C'est-à-dire: est-ce que ces packages sont utilisés? Parce qu'il s'agit de nuxt et du linter, donc ils sont utiles...

Je n'ai pas vu d'utilisations de nuxt sur pix et les linters sont dans chaque projet d'où ma question de leur intérêt à la racine.

Je me demande s'il ne s'agit pas de reliquats

@sbedeau
Copy link
Contributor

sbedeau commented Dec 7, 2020

Avec les outils https://www.npmjs.com/package/npm-check et https://www.npmjs.com/package/depcheck, j'obtiens ça. Même si ça rejoint à peu près mon intuition, j'ai quelques doutes sur leur fiabilité au sein de notre projet

Screenshot 2020-12-07 at 17 25 31

@alexandrecoin
Copy link
Contributor

Avec les outils https://www.npmjs.com/package/npm-check et https://www.npmjs.com/package/depcheck, j'obtiens ça. Même si ça rejoint à peu près mon intuition, j'ai quelques doutes sur leur fiabilité au sein de notre projet

Screenshot 2020-12-07 at 17 25 31

Il ne s'agit pas des dépendances de Pix et non de Pix-site ?

@sbedeau
Copy link
Contributor

sbedeau commented Dec 7, 2020

Avec les outils npmjs.com/package/npm-check et npmjs.com/package/depcheck, j'obtiens ça. Même si ça rejoint à peu près mon intuition, j'ai quelques doutes sur leur fiabilité au sein de notre projet
Screenshot 2020-12-07 at 17 25 31

Il ne s'agit pas des dépendances de Pix et non de Pix-site ?

Ah damn, bien vu ! D'où les confusions 🤦

@bpetetot bpetetot changed the title Montée de version des packages eslint, nuxt et autoprefixer [TECH] Montée de version des packages eslint, nuxt et autoprefixer Dec 8, 2020
@bpetetot bpetetot merged commit 2d470e4 into dev Dec 8, 2020
@bpetetot bpetetot deleted the tech-upgrade-package-versions branch December 8, 2020 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants