Skip to content

Conversation

@arthurbergmz
Copy link
Collaborator

Initial proposal for the installation section in Portuguese.

Needs to be reviewed.


Estou com algumas dúvidas em relação à tradução, @ErickPetru:

  1. Sobre este pedaço da documentação:
- Leaves prod/dev branches with `process.env.NODE_ENV guards` (must be replaced by bundler) 

Entendo que "guards" refere-se a uma função guard que verifica se process.env.NODE_ENV possui um dos valores específicos de ambiente (prod ou dev, por exemplo), antes de se gerar a distribuição necessária para o ambiente informado. Portanto, a uma possível correção seria:

- Leaves prod/dev branches with `process.env.NODE_ENV` guards (must be replaced by bundler) 
  1. Tomei liberdade para adicionar alguns termos ao translations.md, posso realizar o rollback sem problemas, se for desejado ou, então, atualizar algum.

  2. Decidi utilizar distribuição para build, entretanto, a documentação da V2 utiliza _build_. Mantenho assim ou utilizo _build_ também?

Copy link
Member

@ErickPetru ErickPetru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A qualidade está excelente, só recomendo trocar a notação de maiúsculas dos títulos para ficarem mais próximos dos originais em inglês (claro, mantendo as preposições em minúsculas).

@arthurbergmz
Copy link
Collaborator Author

@ErickPetru só me avisar se houver mais alguma mudança a ser realizada ;)

@ErickPetru ErickPetru merged commit 49a4563 into vuejs-br:master Sep 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants