Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: add monitor command #66

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.

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

Merged
lucarin91 merged 16 commits into arduino:main from lucarin91:add-bridge-monitor
Dec 9, 2025
Merged

Conversation

@lucarin91
Copy link
Contributor

@lucarin91 lucarin91 commented Nov 12, 2025
edited
Loading

Motivation

Add a monitor command for interacting with the microcontroller monitor.

Change description

You can test it by:

  1. flashing the monitor examples
  2. Install this version of arduno-app-cli with task board:install
  3. adb shell and run arduino-app-cli monitor

You should see the monitor print from the microcontroller, and you should be able to send in message too.

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@per1234 per1234 added the enhancement New feature or request label Nov 12, 2025
@lucarin91 lucarin91 marked this pull request as ready for review November 28, 2025 18:03
@lucarin91 lucarin91 requested a review from a team December 1, 2025 10:27
@lucarin91 lucarin91 changed the title (削除) feat: add bridge monitor command (削除ここまで) (追記) feat: add monitor command (追記ここまで) Dec 1, 2025
Co-authored-by: Cristian Maglie <c.maglie@bug.st>
lucarin91 and others added 3 commits December 2, 2025 10:50
Co-authored-by: Davide <davideneri18@gmail.com>
Co-authored-by: Davide <davideneri18@gmail.com>
Co-authored-by: Davide <davideneri18@gmail.com>
Copy link
Contributor Author

I will try to add some tests, I did a lot of changes, and I am not sure about myself 😄

@lucarin91 lucarin91 merged commit ed4e465 into arduino:main Dec 9, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cmaglie cmaglie cmaglie approved these changes

@dido18 dido18 Awaiting requested review from dido18

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement the monitor command in app-cli

AltStyle によって変換されたページ (->オリジナル) /