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

doc(tooling): réordonnancement #57

Merged
merged 25 commits into from
Jul 28, 2020
Merged

doc(tooling): réordonnancement #57

merged 25 commits into from
Jul 28, 2020

Conversation

geromegrignon
Copy link
Contributor

supprime la partie sécurité https
introduit le fil rouge avec lighthouse
déplace le tooling après l'installation du service worker

related to #17

geromegrignon and others added 3 commits July 6, 2020 14:52
supprime la partie sécurité https
introduit le fil rouge avec lighthouse
déplace le tooling après l'installation du service worker
précise les moyens d'accéder aux devtools Firefox / Safari
met à jour les images des devtools
@geromegrignon geromegrignon marked this pull request as ready for review July 7, 2020 10:03
@geromegrignon
Copy link
Contributor Author

Les exercices sur les devtools (update service worker / cache first / network first) seront revu avec la refonte des exercices sur les services workers.

@geromegrignon geromegrignon requested a review from jefBinomed July 7, 2020 10:05
Copy link
Contributor

@jefBinomed jefBinomed left a comment

Choose a reason for hiding this comment

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

je sais pas trop si il faut pas mettre en avant npx ?

CLI:

```shell
npm i -g lighthouse
Copy link
Contributor

Choose a reason for hiding this comment

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

est ce qu'on promeut pas plutôt npx ? au lieu d'un npm i -g ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

En effet et en le testant l'exécution est rapide.

Copy link
Contributor

Choose a reason for hiding this comment

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

d'une manière générale, je ne suis plus trop partisant des npm i -g.... j'ai plutôt tendance à pousser l'outil dans la version souhaité dans mes devDependancies et l'utiliser ensuite avec npx

CLI:

```shell
npx lighthouse http://localhost:8080/ --view
Copy link
Contributor

Choose a reason for hiding this comment

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

du coup, pour continuer sur cette discussion, tu pense qu'il faudrait qu'on fige une version de lighthouse dans notre package.json ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

en effet cela permettrait d'être résilient aux changements surtout que la version actuelle correspond à notre besoin.

geromegrignon and others added 14 commits July 13, 2020 17:33
doc(app-shell): suppression slides sur la vitesse d'affichage
Fix format of Run Project section
supprime la partie sécurité https
introduit le fil rouge avec lighthouse
déplace le tooling après l'installation du service worker
précise les moyens d'accéder aux devtools Firefox / Safari
met à jour les images des devtools
@geromegrignon geromegrignon merged commit 9691a26 into master Jul 28, 2020
@geromegrignon geromegrignon deleted the 17-tooling branch July 28, 2020 08:12
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.

3 participants