Repository navigation
chore(upstream-docs): update watched-document baseline - #165
github-actions[bot] wants to merge 1 commit into
Conversation
miravoss26
left a comment
There was a problem hiding this comment.
Bot baseline update for `upstream-docs-state.json`. It refreshes sha256 and content length for three watched docs (Accuenergy Acuvim II manual, Easee observations, Easee ongoing session API) and adds six new watched URLs (HeishaMon MQTT topics, SunSpec model 802, SunSpec Device Information Model 1.1 and others). Only this state file changes.
- Low: the six new URLs point at raw.githubusercontent.com and sunspec.org. This file only records what the job fetched. The source list that adds them lives elsewhere, so it's worth confirming that list was reviewed when it changed.
- No CI checks on this head yet (mergeable is UNKNOWN).
Safe to merge from my read, as far as this file goes. device-drivers isn't on the autonomous merge allowlist, so a human should merge it.
611081b to
30211f0
Compare
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
30211f0 to
f20ae8a
Compare
miravoss26
left a comment
There was a problem hiding this comment.
Reviewed. This updates upstream-docs-state.json only — new/changed sha256 + content_length baselines for the watched upstream docs (heishamon, pixii, remne_p1ib, sungrow, tesla_vehicle additions; refreshed hashes for acuvim, easee, saj, teslamate). No driver code, no logic change.
- Correctness: matches the PR body's own description (accepting new observed bytes as the new baseline). One new entry (
pixii::sunspec-device-info-pdf) hasconsecutive_failures: 1/last_error: 403 Forbidden— that's an honest failure record, not a merge blocker, but worth a human glance if pixii driver work depends on that PDF. - Security: no secrets, no code paths, no new runtime network calls — these are just tracked doc URLs for the watcher, all public, same shape as existing entries.
- Empty status rollup is expected per the PR body (GITHUB_TOKEN-opened PRs don't trigger workflows) — close/reopen to run checks if you want CI's own verdict before merging.
Safe to merge from my read.
A watched upstream document moved, or a fetch failure changed its
count, so
upstream-docs-state.jsonno longer describes what thewatcher sees. Merging this makes the new observation the baseline the
next run compares against.
Read the
upstream-docissues this run opened before merging: thisaccepts the new bytes as normal, and on its own says nothing about
whether the registers a driver decodes by have moved.
The required checks do not start on their own here. A pull request
opened with
GITHUB_TOKENstarts no workflow run, so this one arriveswith an empty status rollup. Close and reopen it to run them. The
validaterun dispatched against the branch head is a verdict you canread in the meantime -- GitHub does not count it toward the rollup.
Opened by the
watch-upstream-docsworkflow.