Skip to content

Conversation

@willmmiles
Copy link
Member

0.15.2 version of #5057.

Add a field to the OTA metadata structure indicating the oldest base version it's safe to install this update from. This provides a clear path forward in case there are incompatibilities, eg. some case (bootloader compatibility) where 0.16.0 cannot be OTA'd safely from 0.15.2, but a transitional 0.15.3 could potentially arrange the necessary groundwork.

Add a field to the OTA metadata structure indicating the oldest base
version it's safe to install this update /from/.  This provides a clear
path forward in case there are incompatibilities, eg. some case
(bootloader compatibility) where 0.16.0 cannot be installed safely from
0.15.2, but a transitional 0.15.3 can arrange the groundwork.
@willmmiles willmmiles added this to the 0.15.2 milestone Nov 9, 2025
@willmmiles willmmiles self-assigned this Nov 9, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 9, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Apply no restrictions in the 0.15 release line.
@netmindz
Copy link
Member

netmindz commented Nov 9, 2025

For code like this, how easily could we have unit tests?
Not a blocker for acceptance, just a query

@willmmiles
Copy link
Member Author

For code like this, how easily could we have unit tests? Not a blocker for acceptance, just a query

Honestly I'm not sure. I know platformio has some framework for adding unit tests but I haven't explored it yet.

@netmindz netmindz merged commit 0d911e9 into wled:0_15_x Nov 9, 2025
20 checks passed
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.

2 participants