Skip to content

Remove OTA metadata structure version block#5214

Merged
netmindz merged 1 commit intowled:0_15_xfrom
willmmiles:0_15-update-structure-check
Mar 8, 2026
Merged

Remove OTA metadata structure version block#5214
netmindz merged 1 commit intowled:0_15_xfrom
willmmiles:0_15-update-structure-check

Conversation

@willmmiles
Copy link
Copy Markdown
Member

0.15 backport of the OTA version check fix from #5213, on the off chance we have an OTA version 3 before 0.16 launches.

If we cap the acceptable version, it becomes impossible to increase it:
older firmwares will reject it.  Instead we must guarantee
backwards compatibility so long as the magic number is the same.
If a breaking change is required in the future, a new magic number
should be used to identify it.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 20, 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.

@willmmiles willmmiles mentioned this pull request Jan 21, 2026
8 tasks
@netmindz netmindz added this to the 0.15.4 milestone Mar 7, 2026
@netmindz
Copy link
Copy Markdown
Member

netmindz commented Mar 7, 2026

Good to merge this too then @willmmiles ?

@netmindz netmindz merged commit 04a935e into wled:0_15_x Mar 8, 2026
20 checks passed
softhack007 pushed a commit to MoonModules/WLED-MM that referenced this pull request Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants