✨PMIPA-5055 Stash changes before update branch on repo_setup #970
Conversation
…bug on git clone of repo_setup of SAL
|
Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN]. 💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform. |
tools/repo_setup.py
Outdated
| ).decode("utf-8") | ||
| if status_output: | ||
| self.run_command( | ||
| ["git", "stash", "push", "--include-untracked"], |
There was a problem hiding this comment.
Sugerância de verbosidade:
| ["git", "stash", "push", "--include-untracked"], | |
| ["git", "stash", "push", "-m", '"STASHED BY REPO_SETUP.PY"', "--include-untracked"], |
Só pro caso do pop falhar
There was a problem hiding this comment.
Eu tinha botado com nome mas não achei uma forma de fazer o pop pelo nome e perdi tempo demais nisso... Vou botar então
There was a problem hiding this comment.
Acho que pode deixar o pop como está... Tava pensando só numa informação a mais se o pop falhar por conta de conflito, por ex...
tools/repo_setup.py
Outdated
| ["git", "stash", "push", "--include-untracked"], | ||
| _path, | ||
| ) | ||
| stash_applied = True |
There was a problem hiding this comment.
Acho que seria interessante pegar o result do comando aqui... Só pra não achar que stashou se não tinha nada pra stashar...
There was a problem hiding this comment.
Mas eu vejo se tem algo pra dar stash antes.
There was a problem hiding this comment.
Tendi... É, não sei se existe um cenário que o comando de stash possa falhar, fora não ter nada pra stashar (já verificado)...
Na hora que ele aplica o stash mostra os arquivos q vieram do stash aplicado |
✨ PMIPA-5055 Stash changes before update branch on repo_setup. Fixes bug on git clone of repo_setup of SAL