Skip to content

Run commands in an atom terminal emulator instead of a buffered process#96

Merged
moxley merged 4 commits intomoxley:masterfrom
nolanpro:master
Jan 27, 2017
Merged

Run commands in an atom terminal emulator instead of a buffered process#96
moxley merged 4 commits intomoxley:masterfrom
nolanpro:master

Conversation

@nolanpro
Copy link
Copy Markdown
Contributor

@nolanpro nolanpro commented Jan 7, 2017

Hey @moxley Thanks for the package, I use it a bunch! Once issue I had was not being able to step into a debug like binding.pry because the command output is a buffered process. I replaced that with making calls to the platformio-ide-terminal package on my fork and it seems to work great. What are your thoughts on this? Thanks!

@moxley
Copy link
Copy Markdown
Owner

moxley commented Jan 9, 2017

mind blown Thank you @nolanpro! This works very well, and it's going to make a lot of people happy.

@moxley
Copy link
Copy Markdown
Owner

moxley commented Jan 9, 2017

@nolanpro: I'll review this over the next several days.

@moxley
Copy link
Copy Markdown
Owner

moxley commented Jan 16, 2017

@nolanpro This is looking really good. What's the key binding for closing the window pane, or canceling the test?

@nolanpro
Copy link
Copy Markdown
Contributor Author

@moxley Thanks! I might need to rework how the commands go through to the terminal. I'll take a look at it this week and update the PR

@nolanpro
Copy link
Copy Markdown
Contributor Author

@moxley I added back in toggle and cancel. Cancel just sends the close command to the terminal plugin which effectively cancels the process. Let me know what you think. Thanks!

@moxley
Copy link
Copy Markdown
Owner

moxley commented Jan 27, 2017

It works! This is exciting. Thank you @nolanpro, for this excellent contribution to ruby-test.

@moxley moxley merged commit 9bcf5e9 into moxley:master Jan 27, 2017
@nolanpro
Copy link
Copy Markdown
Contributor Author

@moxley Thanks! I'll keep an eye on this repo in case any issues come up related to the terminal

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.

2 participants