Skip to content

Conversation

rougeth
Copy link
Collaborator

@rougeth rougeth commented Jan 28, 2020

No description provided.

@rffontenelle
Copy link
Collaborator

Apoio a mensagem ser mais explícita, mas algumas considerações:

  • Sua mensagem de commit está com 51 caracteres. Podemos manter em até 50 ?
  • "Sync ... from ... to" estaria certo? Penso sincronizar como "agora estão ambos iguais", ao passo que estranhei um pouco o "from to" como se fosse um copiar daqui para lá.
  • Se agora aplicamos powrap, não estaria tecnicamente correto falar em sync :)

Que tal "Update translations from Transifex"?

@rougeth
Copy link
Collaborator Author

rougeth commented Jan 28, 2020

Perfeito, faz todo sentido.

@rougeth
Copy link
Collaborator Author

rougeth commented Feb 3, 2020

@rffontenelle posso mergear esse?

@rffontenelle
Copy link
Collaborator

rffontenelle commented Feb 3, 2020

Opa, manda bala. Acho que meu 👍 não foi tão claro. 😄

@rougeth
Copy link
Collaborator Author

rougeth commented Feb 3, 2020

Hahaha achei que vc ia aprovar o PR. E eu não lembrava vc tinha marcado com 👍 depois deu ter mudado.

@rougeth rougeth merged commit bc186a8 into 3.8 Feb 3, 2020
@rougeth rougeth deleted the rougeth-patch-1 branch February 3, 2020 19:34
@rffontenelle
Copy link
Collaborator

Hahaha achei que vc ia aprovar o PR. E eu não lembrava vc tinha marcado com +1 depois deu ter mudado.

My bad. Eu nunca estive numa posição de approver de MR e nem reparei se eu tinha acesso a isso aqui. Ficarei mais atento da próxima vez.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants