search

n7consulting / Incipio

ERP / CRM for Junior-Entreprises.
http://jeyser-crm.n7consulting.fr
GNU Affero General Public License v3.0
42 stars 17 forks source link

Best practices Releases - PR #339

Closed KiDayz closed 3 years ago

KiDayz commented 3 years ago

Hello, @Stoakes J'ai vu que tu avais validé la PR (merci).

Du coup pour avoir les fix disponibles sur notre version du master, que faut-il faire ?

En te remerciant,

Stoakes commented 3 years ago

Merci pour la PR.

Pour avoir le code de la branche master dans votre fork

Tu peux avoir le code d'n7consulting/Incipio:master sur la branche master de votre dépôt, en mettant a jour votre branche master à partir de chez nous. Ce lien indique la marche à suivre.

Par contre comme tu as développé tes modifications sur master, ça va être un peu plus compliqué. A ta place j'effacerais tes commits de master et mettrais à jour depuis le notre. Et pour les prochaines modifications, tu peux développer sur une branche et ouvrir une PR depuis cette branche. Un exemple de ça avec Ouest-insa: ils ont créé une branche avenant-add dans leur dépôt et on ouvert la PR vers notre branche master.

Pour avoir les modifications dans votre version (en production) de Jeyser

Si tu souhaites avoir ces modifications effectives sur la version qui tourne pour votre Junior, c'est possible, avec l'inconvénient que ça fait sortir du cycle de release officielle de Jeyser, mais rien de grave tant qu'un développeur est là pour supporter ça.

Globalement, il va falloir faire comme pour une mise à jour mineure, sauf qu'au lieu de faire quelque chose comme git checkout v3.0.2 (qui va chercher une version taggée du code), il faudra se placer sur la branche master.

En d'autres termes, la mise à jour devrait ressembler à ca (car il n'y a pas de migrations de bases de données):

# Aller dans le dossier d'installation de Jeyser
cd /var/www/Incipio

# Se placer sur la branche master et récupérer la dernière version
git checkout master
git pull --prune

# Construire la nouvelle image pour le container web
docker-compose build web

# Démarrer une nouvelle instance du container web
docker-compose up -d web

On pourrait aussi faire une nouvelle release, mais j'ai l'impression que de nouvelles contributions ne devraient pas tarder donc autant attendre encore un peu.

Digression sur un autre sujet: je ne sais plus si c'est écrit quelque part dans la doc, mais beaucoup de choses sont en anglais pour permettre d'exploiter Jeyser et nos contributions open-source dessus dans nos CV en France comme à l'international.

KiDayz commented 3 years ago

Merci pour les conseils et le mode d'emploi ! En effet, j'ai une PR qui arrive ;)