Skip to content

Feat: estrutura da CLI usando o meow#23

Merged
victorperin merged 5 commits into
oloquinho:masterfrom
victorperin:feat/cli-structure
May 8, 2019
Merged

Feat: estrutura da CLI usando o meow#23
victorperin merged 5 commits into
oloquinho:masterfrom
victorperin:feat/cli-structure

Conversation

@victorperin

@victorperin victorperin commented Apr 29, 2019

Copy link
Copy Markdown
Member

Essa alteração foi maior do que acabei prevendo, pois a maior parte foi estruturação do código. Segue a lista do que fiz:

  • Fiz um refactor na estrutura do código, para não precisar ficar verificando se está ou não no modo cli ou no modo de requisição (facilita testes e simplifica complexidade)
  • Aproveitei para mover alguns arquivos para subpastas, pra simplificar a visualização
  • Adicionei o pacote meow
  • Adicionei o meow no modo cli.

Testado:

  • Windows
  • Mac
  • Linux

Como testar:

  • Clone o repositorio
  • Adicione meu remote: git remote add perin git@github.com:victorperin/oloquinho.git
  • npm install
  • npm link (para instalar global, do repositorio e poder testar)

Teste:

  • oloquinho
  • oloquinho --help
  • oloquinho --version

@victorperin victorperin added the enhancement New feature or request label Apr 29, 2019

@luanorlandi luanorlandi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tem um exemplo para testarmos? Simular um cli com argumento

Vou testar no Windows

Comment thread package.json
Comment thread package.json
@victorperin

Copy link
Copy Markdown
Member Author

Opa @luanorlandi tem sim, podemos usar o --help e --version pois são padrões.

@victorperin

Copy link
Copy Markdown
Member Author

Testei no windows e ta funcionando. Preciso de alguém para testar no Mac.

@luanorlandi

Copy link
Copy Markdown
Member

Nice! Testei aqui também, foi no Windows

@victorperin victorperin merged commit f3fb42b into oloquinho:master May 8, 2019
@victorperin victorperin deleted the feat/cli-structure branch May 8, 2019 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants