7 expand baud rate support and fix macOS CI workflow#8
Merged
Conversation
- Expanded the BAUD_RATES array from 3 to 20 entries, covering standard speeds from 110 bps up to 3 Mbps. - Updated the Default implementation for PortSettings to point to the new index for 115200 bps.
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR addresses two main areas: expanding the supported baud rates for the serial monitor to improve hardware compatibility and fixing a broken macOS build in GitHub Actions.
Changes
Increased Options: Updated the BAUD_RATES array from 3 to 20 supported rates.
Range: Now includes standard low speeds (110, 300, 600) and high-speed rates (up to 3,000,000) common in modern development (ESP32, high-speed sensors, etc.).
Default Settings: Updated the default PortSettings index to 11 to ensure the default remains 115200 despite the array expansion.
macOS Runner Update: Migrated the Intel build runner from macos-13 to macos-15-intel.
Reason: The previous configuration was causing failures in the packaging workflow. Using the specific -intel runner ensures stable builds for the x86_64-apple-darwin target.