Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Para Revisão] - Capítulo #5 - CLI - Run, Serve, Start, V2 #53

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wgrafael
Copy link
Member

@wgrafael wgrafael commented May 4, 2016

Para revisão e finalizar o capítulo de CLI. @juarezpaf

@wgrafael wgrafael changed the title [Para Revisão] - Capítulo 5 - CLI - Run, Serve, Start, V2 [Para Revisão] - Capítulo #5 - CLI - Run, Serve, Start, V2 May 4, 2016
@lazaromenezes lazaromenezes self-assigned this May 10, 2016

| Comando | Descrição |
|---------|-----------|
| ionic run | Executar projeto ionic em um dispositivel conectado |
Copy link
Contributor

Choose a reason for hiding this comment

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

dispositivo conectado

@lazaromenezes
Copy link
Contributor

@wgrafael, adicionei algumas sugestões de atualização para o primeiro arquivo, mas gostaria de adicionar outras duas sugestões, no caso, em geral:

  1. Apesar de ficar um pouco diferente da original, o que acham de colocar um espaço antes e depois do sinal de pipe (|) na primeira coluna das Flags? Acredito que facilitaria a leitura.
  2. O que acham de, ao invés de utilizar os verbos no infinitivo, deixar de uma forma diferente (não sei como classificar isso gramaticalmente 😄) Ex.: Substituir Imprimir logs do console do aplicativo no Ionic CLI (livereload requirido) por _Imprime os logs do console do aplicativo no Ionic CLI (livereload requirido)_

@lazaromenezes lazaromenezes removed their assignment Jul 19, 2016
@lazaromenezes
Copy link
Contributor

@juarezpaf @gabrielclima Este pull request estava atribuído pra mim, mas acho que está parado há algum tempo. Sabem se estes arquivos foram atualizados em outro PR?

@gabrielclima
Copy link

Bom, pela issue desse capítulo, quem está com a tradução completa do CLI é o @wgrafael. Só faltavam essas desse PR realmente... Esperar o @wgrafael fazer as atualizações parar fazer o merge

@@ -0,0 +1,3 @@
# Descriçao #

O comando *v2* ira imprimir informações relevante para iniciar uma aplicação na versão V2.

Choose a reason for hiding this comment

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

irá

Copy link
Member Author

Choose a reason for hiding this comment

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

Opa, desculpe pela demora...
Irei corrigir ainda hoje e já faço o commit.

Em 20 de jul de 2016 11:22 AM, "Gabriel Lima" [email protected]
escreveu:

In chapters/05-cli/5j-v2.md
#53 (comment):

@@ -0,0 +1,3 @@
+# Descriçao #
+
+O comando v2 ira imprimir informações relevante para iniciar uma aplicação na versão V2.

irá


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IonicBrazil/ionic2-docs/pull/53/files/e3a94624f79c06d2862c09feb8ccf4b85969226a#r71533976,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABR4t89MlmCcAOvlTQw4M9xwBXC2GyJ2ks5qXi8_gaJpZM4IWwaL
.

@ednilsonamaral
Copy link
Member

Opa, e aí cara, beleza?!

Por favor, pode retirar do seu PR os commits referente aos sumários, tanto do README.md geral como do capítulo. A partir de agora, não precisa mais linkar nos sumários. Faremos isso separadamente.

Obrigado! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants