Skip to content

Update bootloader options#5

Merged
h2zero merged 3 commits intomasterfrom
dev
Feb 7, 2026
Merged

Update bootloader options#5
h2zero merged 3 commits intomasterfrom
dev

Conversation

@h2zero
Copy link
Copy Markdown
Owner

@h2zero h2zero commented Sep 10, 2025

No description provided.

@h2zero h2zero force-pushed the dev branch 3 times, most recently from 021f830 to 64a5e2b Compare September 10, 2025 21:52
@h2zero h2zero force-pushed the dev branch 4 times, most recently from cdfe371 to 83e8d51 Compare January 21, 2026 02:01
@h2zero h2zero marked this pull request as ready for review January 21, 2026 14:45
@h2zero h2zero requested a review from Copilot January 21, 2026 14:46
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request refactors the bootloader configuration system to use a more flexible array-based structure instead of a dictionary, allowing multiple bootloader options per board. The PR removes explicit bootloader selections from example configurations and adds support for the Seeed XIAO nRF52840 Sense board.

Changes:

  • Refactored bootloader configuration from dictionary to array format in board JSON files, enabling multiple bootloader options per board
  • Updated Python build script to iterate through bootloader arrays and apply selected bootloader settings
  • Removed hardcoded board_bootloader settings from example platformio.ini files, relying on default bootloader selection

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 46 comments.

Show a summary per file
File Description
builder/frameworks/arduino/nrf5.py Updated bootloader handling logic to work with array structure and added default bootloader selection
boards/*.json (multiple) Converted bootloader configs from dict to array format, added bootloader options with detailed parameters
examples/BLE-server-test/platformio.ini Removed explicit board_bootloader settings, added seeed_xiao_nrf52840_sense board
examples/BLE-client-test/platformio.ini Removed explicit board_bootloader settings, added seeed_xiao_nrf52840_sense board
.github/workflows/examples.yml Updated workflow triggers to specific branches and updated actions/checkout version

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@h2zero h2zero changed the title Update bootloader options - WIP Update bootloader options Jan 21, 2026
@h2zero h2zero requested a review from Copilot January 21, 2026 19:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 21, 2026

@h2zero I've opened a new pull request, #7, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 23 out of 23 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@h2zero h2zero merged commit aa100b8 into master Feb 7, 2026
6 checks passed
@h2zero h2zero deleted the dev branch February 7, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants