Skip to content

Conversation

rffontenelle
Copy link
Collaborator

Aplica powrap a todos os arquivos .po do repositório e adiciona execução de powrap logo após baixar traduções de tx pull toda vez que make pull é executado.

A opção --modified faz com que powrap identifique quais arquivos .po foram alterados ou são novos, e atua apenas neles.

@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

@rffontenelle demorei pra revisar e agora estamos com conflitos. Sabe a diferença de tempo rodando o powrap sem o --modified? As vezes nem precisa do argumento.

@rffontenelle
Copy link
Collaborator Author

@rougeth O tempo de execução sem --modified não é nada proibitivo. Achei interessante usá-lo para o script não "tocar" arquivos não envolvidos no commit, mas talvez não haja problema aplicar a todos os arquivos substituindo a linha
$(VENV)/bin/powrap --modified --quiet
por
$(VENV)/bin/powrap --quiet *.po **/*.po

Prefere assim?

@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

Teoricamente, ele não deveria alterar nenhum arquivo que não houve alteração, né? Acho que podemos testar, o que acha?

@rffontenelle
Copy link
Collaborator Author

Bora. Mandei um commit atualizando conforme últimos commits no 3.8, e sem alterações minhas aos arquivos po. Uma vez mesclado, deve aplicar powrap a todos os arquivos do repositório.

@rougeth rougeth merged commit 1abe5c6 into python:3.8 Jan 28, 2020
@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

Show!

@rffontenelle rffontenelle deleted the add-powrap branch January 28, 2020 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants