majeur - Nettoyage des branches

[Gaetanbrl]

Je viens de comparer les branches en cours suite à #111.

Le dernier commit commun entre develop et master est le suivant : 759975c

Il y a pas mal de commit sur develop qui ne sont pas sur master. Ca vaudrait le coup de faire une release @bchartier ?

Voici les étapes à suivre :

1. passer les commits > au 759975c du 06/03/2021 de master à develop (via la commande cherry-pick). Les commits master qui ne sont pas dans develop ne sont pas énormes ca devrait être largement gérable. Utiliser la commande push -f pour pousser les modifs sur develop (on a modifié l'arbre...)
2. developp contenant les commits, on revient sur le commit Le dernier commit commun entre develop et master est le suivant : 759975c
   via un reset --hard 759975c7f2943f308c1488b458441785d006d7a7 && push -f
3. gérer la PR Mise à jour du design #110 en ciblant developp
4. nouvelle version (si souhaité)

[Gaetanbrl]

Etapes 1 et 2 Dispo sur
https://github.com/Gaetanbrl/mviewerstudio/tree/master
https://github.com/Gaetanbrl/mviewerstudio/tree/develop

@bchartier tu valides ?

• cherry-pick master vers develop pour commits > 759975c
• roll back branche master pour se positionner sur commit 759975c
• changer branche cible Mise à jour du design #110 vers develop
• new release

[Gaetanbrl]

@lecault peux-tu vérifier les infos du config-sample.json suivant ?
https://github.com/Gaetanbrl/mviewerstudio/blob/develop/config-sample.json

[lecault]

Il est pas à jour vis à vis du master (notamment les scan ign).

Je pense qu'il faut migrer le fichier du master vers develop ?

[Gaetanbrl]

j'ai fait les cherry-pick entre master et develop, normalement les infos de master sont dans develop.
Comment vois-tu qu'il n'est pas à jour ?

Il faut comparer le config-sample.json de la branche master de geobretagne avec le même fichier de la branche develop de mon dépôt :

https://github.com/geobretagne/mviewerstudio/blob/master/config-sample.json
et
https://github.com/Gaetanbrl/mviewerstudio/blob/develop/config-sample.json

L'objectif est de conserver tous les commits de master et de les migrer vers develop pour ne pas écraser de modifications qui pourrait être dans develop qu'on aurait pas mis dans master.

[Gaetanbrl]

Il est pas à jour vis à vis du master (notamment les scan ign).

Le scan ign c'est plan_ign ?

[Gaetanbrl]

@lecault dans le fichier https://github.com/Gaetanbrl/mviewerstudio/blob/develop/config-sample.json

il y a ces 3 paramètre qui ne sont pas dans master :

https://github.com/Gaetanbrl/mviewerstudio/blob/62896ca09d0d9f83d3206ac66b48a2d58d5e6bce/config-sample.json#L4-L6

Je pense que ca a été apporté dans develop et pas dans master, donc on ne peut pas juste écraser le fichier en remplaçant celui de develop par celui master.

[lecault]

Hum faudra que je prenne le temps de regarder. L'idée est de mettre nickel sur develop puis de pousser sur master ?

[Gaetanbrl]

L'idée est de mettre nickel sur develop puis de pousser sur master ?

Oui car les merge entre la branche developp et master on ramenés des commits qui ne sont pas dans develop.

Comme je le précisais, seuls les commits qui sont après ce commit 759975c sont à passer dans develop (donc seuls les commits après le 6 mars)

[bchartier]

Je suis dispo aujourd'hui pour travailler sur ce ticket.

[bchartier]

Il y a pas mal de commit sur develop qui ne sont pas sur master. Ca vaudrait le coup de faire une release @bchartier ?

Oui, cela vaudrait le coup.
L'idéal serait de faire une release sans les modifs de @HWFord histoire de laisser le temps de tester ces modifs et d'apporter tranquillement des modifs/corrections si besoin d'ici une prochaine release.

[bchartier]

Comme je le précisais, seuls les commits qui sont après ce commit 759975c sont à passer dans develop (donc seuls les commits après le 6 mars)

J'ai intégré la PR #113 dans la branche develop.

[Gaetanbrl]

L'idéal serait de faire une release sans les modifs de @HWFord

Parfait

[bchartier]

L'idéal serait de faire une release sans les modifs de @HWFord

Parfait

Pour la release, on avait défini des règles de numérotation ? On doit garder la cohérence avec la dernière release de mviewer ?

[Gaetanbrl]

Pour la release, on avait défini des règles de numérotation ?

Ca ne me parle pas.

On doit garder la cohérence avec la dernière release de mviewer ?

Pour le code / dépendance oui. Mais on peut avoir une numérotation indépendante pour mviewer studio.

D'ailleurs je ne sais pas s'il y a une question de dépendance dans le readme mviewer ou mviewer studio pour indiquer que telle version studio est valable pour telle version mviewer ?

[Gaetanbrl]

@bchartier merci pour le merge.
Est-ce que je peux faire le roll back pour faire revenir la branche master sur le commit (comme dernier commit) 759975c ?

[bchartier]

Pour la release, on avait défini des règles de numérotation ?

Ca ne me parle pas.

On doit garder la cohérence avec la dernière release de mviewer ?

Pour le code / dépendance oui. Mais on peut avoir une numérotation indépendante pour mviewer studio.

ok

D'ailleurs je ne sais pas s'il y a une question de dépendance dans le readme mviewer ou mviewer studio pour indiquer que telle version studio est valable pour telle version mviewer ?

On a une dépendance car mviewer studio crée un fichier de config pour mviewer mais on ne documente cette dépendance entre versions nulle part je crois. On devrait en parler lors de la prochaine réunion du comité de pilotage à mon avis.

[bchartier]

@bchartier merci pour le merge.
Est-ce que je peux faire le roll back pour faire revenir la branche master sur le commit (comme dernier commit) 759975c ?

Logiquement master devrait pointer sur le dernier commit de la dernière release, non ?
genre : e580fff

[bchartier]

On doit garder la cohérence avec la dernière release de mviewer ?

Pour le code / dépendance oui. Mais on peut avoir une numérotation indépendante pour mviewer studio.

ok

J'ai créé un milestone 3.2 avec quelques tickets/pull requests associés :
https://github.com/geobretagne/mviewerstudio/milestone/2
Merci de me signaler si je dois ajouter d'autres choses en vue de la release 3.2.

[bchartier]

@bchartier merci pour le merge.
Est-ce que je peux faire le roll back pour faire revenir la branche master sur le commit (comme dernier commit) 759975c ?

Logiquement master devrait pointer sur le dernier commit de la dernière release, non ?
genre : e580fff

J'ai sans doute raison en théorie mais en fait je pense qu'on a intérêt à ne pas toucher master tant que la release 3.2 n'a pas été validée.
Ça vous semble raisonnable ?

[lecault]

@lecault dans le fichier https://github.com/Gaetanbrl/mviewerstudio/blob/develop/config-sample.json

il y a ces 3 paramètre qui ne sont pas dans master :

https://github.com/Gaetanbrl/mviewerstudio/blob/62896ca09d0d9f83d3206ac66b48a2d58d5e6bce/config-sample.json#L4-L6

Je pense que ca a été apporté dans develop et pas dans master, donc on ne peut pas juste écraser le fichier en remplaçant celui de develop par celui master.

Effectivement il y a juste une différence sur ces 3 lignes sinon c'est juste des espaces / tabulations qui changent.
Ces 3 fichiers sont présents dans le master et dans develop à un autre endroit.

C'est lié à la conf GeOrchestra.

Un niveau a été ajouté, il faudrait mettre :

    "upload_service": "srv/php/store.php",
    "delete_service": "srv/php/delete.php",
    "list_service": "srv/php/list.php",
    "store_style_service": "srv/phpstore/style.php",

[spelhate]

Je viens de merger develop dans master.
La CI pour fabriquer les images docker fonctionne bien maintenant

Merci @pierrejego @bchartier @lecault @Gaetanbrl