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

Add argument to monitor without building/flashing #197

Closed
SergioGasquez opened this issue Jun 23, 2022 · 3 comments · Fixed by #205
Closed

Add argument to monitor without building/flashing #197

SergioGasquez opened this issue Jun 23, 2022 · 3 comments · Fixed by #205
Assignees
@SergioGasquez
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@SergioGasquez
Copy link
Member

Add an argument that opens the serial monitor without having to go through building and flashing.
@SergioGasquez SergioGasquez added the enhancement New feature or request label Jun 23, 2022
@jessebraham jessebraham moved this to Todo in esp-rs Jun 23, 2022
@MabezDev
Copy link
Member

This would be useful, perhaps specifying espflash --monitor with no elf file should be a valid option. Currently it just returns the help text.

@jessebraham
Copy link
Member

Might have to be a subcommand to make it consistent between both binaries. cargo-espflash will just build and flash the project if no subcommand is provided.

@jessebraham jessebraham added the good first issue Good for newcomers label Jul 4, 2022
@SergioGasquez SergioGasquez self-assigned this Jul 6, 2022
@SergioGasquez SergioGasquez mentioned this issue Jul 6, 2022
Merged
Repository owner moved this from Todo to Done in esp-rs Jul 6, 2022
@jessebraham
Copy link
Member

Thanks for doing this @SergioGasquez!

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

@SergioGasquez SergioGasquez

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
esp-rs
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add serial-monitor subcommand
3 participants
@jessebraham @MabezDev @SergioGasquez