Skip to content

fix: add disable-releaser: true to autolabeler job - #420

Merged
tomer-w merged 3 commits into
mainfrom
copilot/fix-mppt-pv-current-issue
May 29, 2026
Merged

fix: add disable-releaser: true to autolabeler job#420
tomer-w merged 3 commits into
mainfrom
copilot/fix-mppt-pv-current-issue

Conversation

Copilot AI commented May 29, 2026

Copy link
Copy Markdown
Contributor

The autolabeler job was failing with Resource not accessible by integration because release-drafter was attempting to create a draft release even on pull_request events, which requires contents: write — a permission the job intentionally restricts to contents: read.

Added disable-releaser: true to the autolabeler step so it only applies labels and skips release creation entirely.

Copilot AI linked an issue May 29, 2026 that may be closed by this pull request
1 task
The /Pv/x/I topics are deprecated in Venus OS. Add a derived formula
metric that calculates PV current as I = P / V for:
- Solarcharger total PV current (Yield/Power / Pv/V)
- Solarcharger per-tracker current (Pv/{tracker}/P / Pv/{tracker}/V)
- Multi RS Solar per-MPPT current (Pv/{n}/P / Pv/{n}/V)
Copilot AI changed the title [WIP] Fix bug where MPPT PV current is not displayed Add derived MPPT PV current metric (I = P/V) May 29, 2026
Copilot AI requested a review from tomer-w May 29, 2026 15:24
@tomer-w
tomer-w marked this pull request as ready for review May 29, 2026 15:49
Copilot AI requested a review from tomer-w May 29, 2026 16:31
Copilot AI changed the title Add derived MPPT PV current metric (I = P/V) fix: add disable-releaser: true to autolabeler job May 29, 2026
@tomer-w
tomer-w merged commit 3581f54 into main May 29, 2026
12 of 13 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.

[Bug]: I don't see MPPT PV current.

2 participants