search

mozfr / firefoxosminisite

Mini-site pour le lancement de Firefox OS en France
Mozilla Public License 2.0
6 stars 8 forks source link

Intégration script de màj du site dans le code #34

Closed pascalchevrel closed 6 years ago

pascalchevrel commented 9 years ago

Le site est mis automatiquement à jour sur chaque commit sur master via un script sur le serveur. Une fois la PR 33 intégrée, il faudra déplacer ce script dans le dossier web, lui faire lancer une commande 'composer update' et lui faire vider le dossier cache. Puisque ce script commence à devoir faire plus de choses que prévues initialement, je vais le mettre dans le source du projet pour référence et pour que d'autres puissent le faire évoluer si nécessaire.

pandark commented 9 years ago

Une remarque un peu plus sérieuse que mon commentaire sur la pull request… Il faudra prendre en compte le cas où l'historique n'est pas linéaire à cause d'une modification faite directement sur le serveur, d'un push --force ou autre : est-ce qu'on laisse les mises à jour bloquées (dans ce cas là, ça serait pratique d'avoir une alerte par mail en cas de problème ou quelque chose comme ça) ou est-ce qu'on pull --force au risque de défaire une modification faite directement sur le serveur ? Il y a peut-être d'autres solutions auxquelles je n'ai pas pensé.

pascalchevrel commented 9 years ago

c'est simple, on ne fait pas de modif directement sur le site :)

pandark commented 9 years ago

Je sais bien, mais je ne sais pas d'où venait celle de la dernière fois (le .gitignore supprimé d'un sous répertoire) qui a bloqué les mises à jour pendant plusieurs mois sans qu'on ne s'en aperçoive. Enfin si tu penses qu'il n'y aura plus de problèmes de ce côté là, je veux bien te faire confiance.

pascalchevrel commented 9 years ago

Le script est en place, il faudrait faire un ou deux tests avec des PR bidons qui ajoutent des dépendances par exemple pour vérifier que la màj se fait bien