Skip to content
This repository was archived by the owner on Jun 3, 2026. It is now read-only.

feat(serial): add comprehensive baud rate selection menu - #42

Open
Adithyakp86 wants to merge 1 commit into
Vaishnav-Sabari-Girish:mainfrom
Adithyakp86:main
Open

feat(serial): add comprehensive baud rate selection menu#42
Adithyakp86 wants to merge 1 commit into
Vaishnav-Sabari-Girish:mainfrom
Adithyakp86:main

Conversation

@Adithyakp86

Copy link
Copy Markdown
Contributor

#35

Which issue does this PR close?

  • Closes # (if applicable)

Rationale for this change

When starting the serial monitor, providing standard baud rate options prevents users from entering invalid values that cause communication failures or display garbage data. This change ensures a more robust and user-friendly experience by replacing/enhancing the selection menu with a comprehensive list of standard rates.

What changes are included in this PR?

  • Added more standard baud rates to the selection menu: 600, 14400, and 28800.
  • Updated main.sh to include the expanded list in the gum choose menu for the Serial Monitor.
  • Synchronized bin/aci.sh with the same baud rate selection improvements for consistency across different installation methods.

Are these changes tested?

Yes, manual verification was performed:

  • Verified that the gum choose menu displays the new baud rate options.
  • Confirmed the arduino-cli monitor command correctly receives the selected baud rate.
  • Checked that selecting a rate works and the UI returns to the main menu correctly after closing the monitor.

Are there any user-facing changes?

Yes. The "Serial Monitor" menu now presents a more complete list of standard baud rates for selection.

@github-actions

github-actions Bot commented Mar 7, 2026

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant