ci : dockerfile

[Lelievre-david]

No description provided.

[sonarqubecloud]

Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

[Tao-Galasse]

LGTM sur cette PR, c'est cool aussi d'avoir pensé à mettre à jour le README :)

C'est dommage cependant de ne pas avoir suivi vos propres standards de code quality et d'avoir mergé la PR malgré le fail de SonarQube

[Tao-Galasse]

le db:create et db:seed ne sont généralement qu'à exécuter une seule fois, lors du tout premier déploiement de l'application ; pas sûr qu'il faille le mettre dans le entrypoint générique qui sera lu à chaque redéploiement

[MisterOryon]

Faute de temps, nous avons réalisé cette PR rapidement pour pouvoir déployer l'application pour la soutenance.
Néanmoins, tu a raison, le db:seed ne devrait être mise en œuvre qu'une seule fois.
Est-il possible de dire a rails de lancer db:seed uniquement s'il n'y a pas de données?
Car sans le db:seed, nous ne pouvons pas nous connecter car il n'y a pas de compte par défaut.

[Tao-Galasse]

Faute de temps, nous avons réalisé cette PR rapidement pour pouvoir déployer l'application pour la soutenance. Néanmoins, tu a raison, le db:seed ne devrait être mise en œuvre qu'une seule fois. Est-il possible de dire a rails de lancer db:seed uniquement s'il n'y a pas de données? Car sans le db:seed, nous ne pouvons pas nous connecter car il n'y a pas de compte par défaut.

@MisterOryon Bien sûr, je comprends bien les impératifs de temps 👍

Je ne pense pas vraiment qu'on puisse lancer la commande de manière conditionnelle, mais au pire, tu peux toujours adapter le contenu du fichier de seed lui-même : rajouter une instruction du genre return if User.first.present? au début du fichier par exemple, ou quelque chose du genre :)

Ou un truc un peu plus spécifique, comme utiliser des find_or_initialize_by / find_or_create_by, par exemple :

User.find_or_initialize_by(email: 'admin@uniser.com').update(first_name: 'Tao')