Skip to content

Document CI workflows in README + workflows reference - #5107

Merged
btshrewsbury-viam merged 3 commits into
mainfrom
docs/ci-workflows-readme
Jun 29, 2026
Merged

Document CI workflows in README + workflows reference#5107
btshrewsbury-viam merged 3 commits into
mainfrom
docs/ci-workflows-readme

Conversation

@btshrewsbury-viam

Copy link
Copy Markdown
Collaborator

What

Documents the repo's GitHub Actions CI as part of an audit of the current jobs (prompted by the long-broken Test Code Samples job).

  • README.md — new Continuous integration section with a quick-reference table of all 13 workflows: what each does, when it runs, and whether it blocks PRs.
  • .github/workflows/README.md — new full reference: per-workflow purpose, triggers, blocking behavior, secrets, and known concerns; a description of the Python helper scripts; the shared Viam test-org dependency; and recurring maintenance themes.

Docs only — no workflow behavior changes.

Job review summary

Reviewed all 13 workflows. Highlights for follow-up (captured in the reference):

Blocking PR checks: vale-lint (prose), codespell, run-htmltest-local (internal links). markdown-lint, prettier-lint, python-lint are informational only (continue-on-error: true).

Scheduled jobs (fail → Jira ticket, don't block):

  • test-code-snippets.yml — Mondays 09:00 UTC; runs Python/Go/TS samples against a live Viam org. Failing continuously since 2025-11-10 (Python step aborts Go/TS). Root cause: the test org has no roleless member for fleet-management-api-orgs.py. Tracked in Fix Test Code Samples: select a non-owner member in fleet-api orgs sample #5106.
  • check-methods.yml — Wednesdays 10:00 UTC; scrapes SDK doc sites for API-method coverage. Job-level continue-on-error: true, so its only failure signal is a Jira ticket.
  • run-htmltest.yml — Tuesdays 10:00 UTC; external link check.

Stale / dead logic worth a cleanup pass (not changed here):

  • Old action versions throughout (checkout@v2/v3, setup-python@v4, github-script@v6); two wjdp/htmltest-action@master moving-branch pins.
  • alias-reminder.yml uses disabled ::set-output → reminder never posts.
  • pr-labeler.yml gates on synchronize/reopened but only subscribes to opened.
  • vale-lint.yml / python-lint.yml carry dead Python-3.8 venv steps.

Verification

  • prettier --check and markdownlint pass on both files.
  • Vale and make build-prod only cover site content under docs/; these are repo/.github/ reference files, so those gates don't apply.

🤖 Generated with Claude Code

Add a Continuous integration section to the top-level README with a
quick-reference table of all 13 GitHub Actions workflows (what each does,
when it runs, and whether it blocks PRs), linking to a new
.github/workflows/README.md that fully describes each workflow: purpose,
triggers, blocking behavior, secrets, and known maintenance concerns.

The reference also documents the shared Viam test-org dependency used by the
scheduled code-sample and SDK-coverage jobs, and collects recurring
maintenance themes (stale action versions, dead PR-automation logic, reliance
on live external services) surfaced while reviewing the jobs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 7ca23b7
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/6a3ed59ff6dc1a0008065108
😎 Deploy Preview https://deploy-preview-5107--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 40 (🟢 up 3 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label Jun 26, 2026
btshrewsbury-viam and others added 2 commits June 26, 2026 13:27
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Add a "TODO: repairs" section at the bottom of the workflows README with an
actionable checklist for fixing the broken/degraded jobs: restoring failure
visibility (the Jira notification steps), the test-code-samples test-org
dependency and step isolation, the dead alias-reminder and pr-labeler logic,
the check-methods signal, and the batchable action-version/cleanup items.

Also close up spaced em/en dashes throughout the file to satisfy the
Viam.DashesSpaces vale rule (level: error).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@btshrewsbury-viam
btshrewsbury-viam merged commit 3c90962 into main Jun 29, 2026
13 checks passed
@btshrewsbury-viam
btshrewsbury-viam deleted the docs/ci-workflows-readme branch June 29, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants