search

nguyenchr / atom-cargo-test-runner

Cargo test runner for Atom Editor
https://atom.io/packages/cargo-test-runner
MIT License
5 stars 3 forks source link

Add 'build' and 'run' commands support #2

Open deszip opened 9 years ago

deszip commented 9 years ago

Would be great to have not only test commands but build and run for target also.

nguyenchr commented 9 years ago

sorry for the late reply I really like the idea of adding the build command not sure about running the program in atom, but I will certainly think about it and see if it is viable

deszip commented 9 years ago

I've quickly build it by just copying your 'test' code and renaming command to build (and changing cargo command to 'cargo run' which calls 'build'). Works great, I can see cargo and my binary output in atom. I just can't get it right because I'm completely new to atom and it's plugin system.

nguyenchr commented 9 years ago

awesome!! what problem are you having exactly?

deszip commented 9 years ago

Problem is that I've just copy-pasted code for test command and renamed it to 'build'. This definitely should be done differently to avoid code duplication^ now it looks awful :)

You can look at my version here: https://github.com/deszip/atom-cargo-test-runner/commits/master

nguyenchr commented 9 years ago

sorry for the lack of activity on this one, it's been a busy few months, I've started refactoring your changes to support the build command, hopefully be finished in the next few days