Prevent null string error for log#50
Conversation
| return false; | ||
| } | ||
| log::add(__CLASS__, 'debug', $result); | ||
| log::add(__CLASS__, 'debug','Result : '. $result); |
There was a problem hiding this comment.
Ajouter la string "Result :" n'aide en rien à fixer quoi que ce soit;
le fix était déjà fait avec le test au-dessus, que j'ai commit ici : 21b7e7d
Et pourquoi avoir commit directement en stable? il y a un conflit entre les 2 version à présent !
There was a problem hiding this comment.
Je reprendrais ta modif effectivement
J'ai du commit en stable, pour régler le souci chez les utilisateurs. Le souci étant que la beta, perd l'usage de la 4G, et pour le moment mes collègues n'ont pas encore trouvé comment regler cela, donc on ne peut pas merge la beta sur la stable.
There was a problem hiding this comment.
Alors il restait la solution de "cherry pick" le commit nécessaire de la beta et de le mettre sur stable, ca évite les conflits de merge après car les lignes (le diff) sont exactement les mêmes
Avec cet exemple:
git checkout master
git pull
git cherry-pick 21b7e7dbe74b3c5ebbf5dfd41e9351bcb17c0a7f <-- le commit qu'on veut qui est actuellement sur la branche beta
git push origin master
mais là il faudrait p-e envisager de "nettoyer" la branche beta si elle pose problème?
=> extraire la feature ou les changements posant problèmes dans un branche dédiée le temps de l'investigation pour pouvoir retrouver une beta qu'on peut merge sans risque sur stable lorsque nécessaire?
3 participants
No description provided.