[Tech] Active le mode strict pour React #1862
Conversation
|
| @@ -14,6 +14,10 @@ on: | |||
| schedule: | |||
| - cron: "38 11 */3 * *" | |||
|
|
|||
| concurrency: | |||
There was a problem hiding this comment.
Je voulais ajouter ça côté Fish aussi ^^
| @@ -55,6 +59,7 @@ jobs: | |||
| fi | |||
| echo $VERSION | |||
| echo "VERSION=$VERSION" >> $GITHUB_OUTPUT | |||
| timeout-minutes: 30 | |||
There was a problem hiding this comment.
Je ne connaissais pas cette prop, bonne idée !
| zoom: 6 | ||
| }) | ||
| }) | ||
| initialMap.setTarget(mapElement.current) |
There was a problem hiding this comment.
Bonne idée de le sortir. Je me demande juste si on n'a pas de double-listening comme il y a un "double first render" en mode strict.
= n"y a-t-il pas besoin de mettre une ref isFirstRenderRef ?
There was a problem hiding this comment.
👍. Il me semble que mettre les dépendances vides sur le useEffect va se déclencher qu'une seule fois au premier render.
| zoom: 6 | ||
| }) | ||
| }) | ||
| initialMap.setTarget(mapElement.current) |
There was a problem hiding this comment.
👍. Il me semble que mettre les dépendances vides sur le useEffect va se déclencher qu'une seule fois au premier render.
Activation du strict mode + ajout de timeouts sur les jobs + annulation des workflows déjà en cours au push
Related Pull Requests & Issues