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

feat: passage en version 8.1 de php + ajout de node, npm et yarn #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tonnevillec
Copy link

No description provided.

@tonnevillec
Copy link
Author

Bonjour Yoan,
Je ne peux pas choisir une autre branche que master pourfaire la PR, donc je propose cette feature si ca t'interesse pour avoir php8.1 ainsi que node + npm et yarn directement au build (toujours pratique pour un projet Symfony ;) )
Bonne continuation

@yoanbernabeu
Copy link
Owner

Bonjour Yoan, Je ne peux pas choisir une autre branche que master pourfaire la PR, donc je propose cette feature si ca t'interesse pour avoir php8.1 ainsi que node + npm et yarn directement au build (toujours pratique pour un projet Symfony ;) ) Bonne continuation

Merci pour la PR !
C'est ok pour NPM et Yarn, cela simplifiera l'utilisation en effet.

Hier lors de mes tests avec PHP 8.1 j'ai eu quelques problèmes (comme dans cette issue : symfony/symfony#44346)

Au choix :

  • on attend
  • ou on propose un docker-compose.override.yml avec php 8.1 ?

Qu'est-ce que tu en penses ?

@tonnevillec
Copy link
Author

Effectivement je viens de faire le test et j'ai le même problème.
On peut mettre en attente la version 8.1 et je vais faire une autre PR avec la v8.0 qui contiendra déjà NPM et Yarn

@yoanbernabeu
Copy link
Owner

Effectivement je viens de faire le test et j'ai le même problème. On peut mettre en attente la version 8.1 et je vais faire une autre PR avec la v8.0 qui contiendra déjà NPM et Yarn

Impeccable.
Si tu arrives à mettre cette PR en Draft ce serait tip top <3

@tonnevillec tonnevillec marked this pull request as draft December 1, 2021 13:21
@tonnevillec tonnevillec marked this pull request as ready for review December 8, 2021 10:02
@tonnevillec
Copy link
Author

les packages laminas ont été mis à jour et le passage en 8.1 est maintenant possible.

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.

2 participants