Repository navigation
A way to ensure everyone is testing locally with the right version of cpython. - #1050
Conversation
|
Pour le coup y'a un avantage à cette version aussi, dans la série "on ne touche pas au clone de l'utilisateur", c'est qu'on ne crée plus le dossier locale dans son clone, et qu'on ne build plus la doc dans son clone, donc on y touche encore moins qu'avant. |
vpoulailleau
left a comment
vpoulailleau
left a comment
There was a problem hiding this comment.
Pour moi, c'est bon, à part un truc que j'ai pas compris.
|
Quelques commentaires rajoutés (pas beaucup), un peu de refacto (beaucoup), et l'utilisation des worktree dans le make merge. Cool :
|
8ef9e67 to
a8d56e3
Compare
vpoulailleau
left a comment
vpoulailleau
left a comment
There was a problem hiding this comment.
LGTM. Last point is to register a bot for auto-bumping 😉
|
Need an approval to merge :] |
vpoulailleau
left a comment
vpoulailleau
left a comment
There was a problem hiding this comment.
Je retente la manip' mais j'imagine qu'il faut le commit bit pour débloquer le merge.
C'était un bon test :) mais en effet, ça ne m'autorise pas à merger :( |
|
Je relis ca et je valide (ou pas 😛 ) |
Avant dans .travis-ci.yml on donnait le SHA1 cpython qui avait donné nos
.po.Mais pour les traducteurs, un
make allutilisait juste leur version de cpython, et pouvait donc ne pas utiliser exactement le bon commit, et tomber sur des erreurs qui ne sont pas de son fait.Avec cette version, on s'assure que tout le monde, pas que Travis, utiliser exactement le bon commit.
closes #989
PS : Je fais bien attention à ne pas toucher au repo cpython de l'utilisateur, je me permet juste un
git fetch, mais pas question depull,checkoutou quoi que ce soit, ça se trouve il est en train de travailler sur ce repo, on ne peut pas y toucher.