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.

Sign up for GitHub

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

Jump to bottom

Corrige abrir no VSCode para mac + terminal #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Corrige abrir no VSCode para mac + terminal #15

wants to merge 1 commit into from

Conversation

nidib
Copy link

@nidib nidib commented Jul 30, 2019

Corrige 'Abrir no VSCode', app compilado não rodava. Fix #14
Adiciona 'Abrir no terminal' apenas para mac (por enquanto). Fix #3

luizjr
luizjr reviewed Jul 30, 2019
View reviewed changes
Copy link

@luizjr luizjr left a comment

Choose a reason for hiding this comment

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

Gostei bastante da melhoria, mas seria possível também poder escolher o terminal(programa) eu mesmo uso o Hyper.
É apenas uma questão é claro, sei que o nativo vai atender para quase todos os casos, meu caso seria a exceção.

@nidib
Copy link
Author

nidib commented Jul 30, 2019

Concordo! Eu mesmo uso o iTerm. Seria interessante ao invés de ficar criando submenus para configurar apps padrões, criar uma janela só pra isso. Vejo a tray muito limitada a opções

@luizjr
Copy link

luizjr commented Jul 30, 2019

Concordo! Eu mesmo uso o iTerm. Seria interessante ao invés de ficar criando submenus para configurar apps padrões, criar uma janela só pra isso. Vejo a tray muito limitada a opções

Nesse contexto eu penso o mesmo, eu até cheguei a adicionar um Menu de Settings, onde chamaria um dialog ou Janela para configurar isso.
Entre outras coisas.

@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

Massa! Acho que devíamos dividir isso em duas PR's, uma feita pra parte de terminal e outra só pra corrigir o problema no Mac. Vou testar o cross-spawn nos três ambientes e aí conforme for aceito a PR aqui e dou um merge.

@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

Fiz um teste usando o spawn nativo com algumas opções a mais aqui #20, podes testar pra ver se funciona pra você?

@nidib
Copy link
Author

nidib commented Jul 30, 2019

@diego3g Funcionando agora...

@nidib nidib closed this Jul 30, 2019
@nidib nidib deleted the my-version branch July 30, 2019 22:34
@nidib nidib restored the my-version branch July 30, 2019 22:34
@nidib nidib reopened this Jul 30, 2019
@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

@nidib Você fechou e reabriu a PR, chegou a testar a PR que eu enviei?

@nidib
Copy link
Author

nidib commented Jul 30, 2019

@diego3g se for a feature/fix-spawn-multiplatform. Sim, está funcionando legal!

@nidib
Copy link
Author

nidib commented Jul 30, 2019

@diego3g Porém só testei no mac

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

@luizjr luizjr luizjr left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cross-spawn não executa no app compilado para mac Adicionar opção para abrir pasta no terminal
3 participants
@nidib @luizjr @diego3g