Require reboot when changing battery parameters

[Forceu]

This PR requires a reboot when changing battery parameters.

Since the GUI doesn’t trigger a reboot, I initially assumed the changes would apply without one. However, I had to unplug my drone for the new settings to take effect.

Summary by CodeRabbit

• Bug Fixes
  • Updated battery management system reboot configuration to improve system stability and handling of power-related scenarios.

[coderabbitai]

Walkthrough

The reboot field in the battery.lua module's exported configuration table is changed from false to true. This is a single-file configuration change with no other logic modifications.

Changes

Cohort / File(s)	Summary
Battery configuration update src/SCRIPTS/BF/PAGES/battery.lua	Changed reboot flag from false to true in the returned module configuration table

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• betaflight/betaflight-tx-lua-scripts#522: Introduced the reboot field as false; this PR changes it to true.

Suggested reviewers

• haslinghuis
• klutvott123

Poem

🐰 A flag flips bright, from false to true,
The battery page knows what to do,
Reboot when ready, steady and sure,
Configuration complete, the fix is pure! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "Require reboot when changing battery parameters" directly corresponds to the main change in the changeset, which modified the reboot flag from false to true in the battery.lua module. The title is concise (6 words, 47 characters), clear, and free of noise or vague terminology. It accurately summarizes the primary change in a way that would be immediately understood by someone reviewing the repository history.
Description Check	✅ Passed	The pull request description adequately explains the change and provides context for why it was necessary. The author clarifies that the reboot flag modification is required because battery parameter changes do not take effect without a physical reboot, despite the GUI not explicitly triggering one. While the description is relatively brief, it conveys the essential information about the change's purpose and the practical experience that motivated it, meeting the baseline requirements for a pull request description in this repository.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 44936c8 and ff383cc.

📒 Files selected for processing (1)

• src/SCRIPTS/BF/PAGES/battery.lua (1 hunks)

🔇 Additional comments (1)

src/SCRIPTS/BF/PAGES/battery.lua (1)

26-26: Battery reboot requirement is appropriate and consistent with codebase patterns.

The verification shows that reboot = true is selectively applied to configurations with actual firmware requirements: battery.lua (now updated), pwm.lua, and failsafe.lua. Other modules retain reboot = false by design, and some (pid_advanced.lua, filters2.lua) use conditional logic based on specific parameter changes. The change aligns correctly with the established pattern—no updates needed elsewhere.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[rccoln]

谢谢更新！期待PR处理完，有新标题生成功能好棒~ ​ 和而不同 群而不党 ---- Replied Message ---- | From | ***@***.***> | | Date | 10/19/2025 17:37 | | To | betaflight/betaflight-tx-lua-scripts ***@***.***> | | Cc | Subscribed ***@***.***> | | Subject | Re: [betaflight/betaflight-tx-lua-scripts] Require reboot when changing battery parameters (PR #531) | coderabbitai[bot] left a comment (betaflight/betaflight-tx-lua-scripts#531) Note Currently processing new changes in this PR. This may take a few minutes, please wait... Commits Reviewing files that changed from the base of the PR and between 44936c8 and ff383cc. Files selected for processing (1) src/SCRIPTS/BF/PAGES/battery.lua (1 hunks) _______________________________________ < We're gonna need a bigger bug zapper. > --------------------------------------- \ \ \ \ /\ ( ) .( o ). Tip CodeRabbit can generate a title for your PR based on the changes. Add @coderabbitai placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the reviews.auto_title_placeholder setting. ✨ Finishing touches Generate docstrings 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 X Mastodon Reddit LinkedIn Comment @coderabbitai help to get the list of available commands and usage tips. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

@rccoln: Thank you for your enthusiasm! 🎉

I'm glad you're excited about the auto-title generation feature. You can use it by adding @coderabbitai placeholder anywhere in your PR title, and CodeRabbit will automatically replace it with a descriptive title based on the changes.

Feel free to reach out if you have any questions or need assistance with this PR or any other reviews!