Skip to content

docs: refresh verified ISO screenshots - #642

Merged
castrojo merged 1 commit into
mainfrom
automation/iso-verification
Oct 10, 2026
Merged

castrojo merged 1 commit into
mainfrom
automation/iso-verification

Conversation

@github-actions

@github-actions github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Screenshots from the existing LUKS ISO test, after every configured flavor passed.
Source commit: 88694e9
Build: https://github.com/projectbluefin/utah/actions/runs/37999779480
ISO E2E: https://github.com/projectbluefin/utah/actions/runs/38001675278
Exact image reference and ISO checksum are in docs/verification/.

@github-actions
github-actions Bot force-pushed the automation/iso-verification branch from 087c40e to 166e1ce Compare October 9, 2026 19:31

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from correctness, security, intent-alignment, style, docs-currency.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@github-actions
github-actions Bot force-pushed the automation/iso-verification branch from 166e1ce to fc438fa Compare October 9, 2026 22:28

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed at head fc438fa.

style

  • (low) docs/verification/README.md:59 reintroduces a trailing blank line at EOF that #623 removed to satisfy the end-of-file-fixer pre-commit hook (.pre-commit-config.yaml:8). Root cause is the generator heredoc at iso/scripts/luks-e2e.sh:951 (blank line before EOF), so every refresh PR will regress this until that line is dropped. Not blocking; this bot branch runs no checks anyway.

No findings from: correctness, security, intent-alignment, docs-currency.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Hive filed out-of-scope review findings as backlog issues so they do not block this PR:

Cap: 3 backlog issue(s) per PR.

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from correctness, security, intent-alignment, style, docs-currency.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@github-actions
github-actions Bot force-pushed the automation/iso-verification branch from 7a0aeae to 37cdfc9 Compare October 9, 2026 23:59

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from correctness, security, intent-alignment, style, docs-currency.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@castrojo castrojo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Routine automated screenshot and provenance refresh.

@castrojo
castrojo merged commit 9faf27e into main Oct 10, 2026
@castrojo
castrojo deleted the automation/iso-verification branch October 10, 2026 01:20
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.

1 participant