Fixed the new run command #147
Conversation
src/Service.php
Outdated
| 'php-etl/console-state:*', | ||
| 'php-etl/workflow-console-runtime:*', | ||
| 'psr/log:*', | ||
| 'monolog/monolog:*', | ||
| 'symfony/dotenv:^6.0', | ||
| 'symfony/console:^6.0', | ||
| 'symfony/dependency-injection:^6.0', |
There was a problem hiding this comment.
Ces dépendances ne sont pas toutes justifiées. Cette modification fonctionne exclusivement dans le cas où on compile sur du filesystem.
Je doute que ça puisse fonctionner correctement dans un cas de build Docker ou Cloud. Au mieux, ça ajoute des dépendances qui n'ont pas à être présentes dans ces 2 cas. Ça pourra provoquer des effets de bord lors du build, ou lors de l'exécution.
Les dépendances Symfony, Monolog et php-etl/console-state sont supposées être requises par le runtime ou un plugin.
There was a problem hiding this comment.
J'ai build et exécuter un satellite en utilisant docker et ça fonctionne également.
De plus, actuellement il n'existe rien pour gérer les dépendances en fonction du runtime ?
There was a problem hiding this comment.
Le problème n'est pas de savoir si dans ton cas aucune erreur n'apparait. Le problème est d'avoir des dépendances obligatoires qui vont empêcher d'en installer d'autres souhaitées par l'utilisateur ou par le runtime Cloud.
|
Il reste 2 points a régler sur cs-fixer https://github.com/php-etl/satellite/actions/runs/7544116483/job/20536686963?pr=147 |
No description provided.