search

Zenika-Training / zenika-formation-framework

Tools to build and present our training material
8 stars 25 forks source link

chore(#130): deploy bin script #131

Closed hgwood closed 7 years ago

hgwood commented 7 years ago

Cette PR remplace le script gcloud-deploy.sh par un équivalent en JS, ce qui permet au framework de l'exposer en tant qu'exécutable aux formations qui dépendent de lui. Elle peuvent alors l'utiliser dans leurs scripts npm. Ceci permet aux formations de ne plus avoir à appeler gcloud-deploy.sh manuellement, ce qui pose des problèmes de permissions (cf #130).

gmembre-zenika commented 7 years ago

En supprimant le script gcloud-install.sh, comment installes tu les binaires gcloud désormais ?

hgwood commented 7 years ago

J'ai supposé que seul Circle CI utilise ces scripts. Or Circle CI a déjà les binaires pré-installés. Ai-je mal supposé ?

Dans tous les cas le SDK gcloud ne semble pas fait pour être installé par projet, mais globalement sur le poste (la preuve en est qu'il est distribué sous forme d'installeur pour Windows), donc il me semble qu'il n'y a pas lieu d'avoir ce script : si un développeur veut le SDK gcloud pour déployer une formation, il l'installe sur son poste en suivant les instructions de Google.

@jlandure peux-tu confirmer ?