Skip to content

docs(skills): drop stale ci.yml references from doc-sync-patch - #419

Merged
danibsheehan merged 1 commit into
mainfrom
docs/doc-sync-patch-ci-yml
Oct 12, 2026
Merged

danibsheehan merged 1 commit into
mainfrom
docs/doc-sync-patch-ci-yml

Conversation

@danibsheehan

Copy link
Copy Markdown
Owner

Summary

  • Why: the doc-sync-patch skill's description (which decides when the skill triggers) and its anti-patterns still named .github/workflows/ci.yml, a file that doesn't exist. After ci: single-source the Go version from go.mod, guard the Dockerfile pin #417 the Go version also no longer lives in a workflow, so the pointer was doubly wrong.
  • What: two lines in .claude/skills/doc-sync-patch/SKILL.md now point at backend/Dockerfile, which check_stack_docs.py reads alongside go.mod and .nvmrc.
  • Not in this PR: the same stale ci.yml reference in the adr-doc-sync-check, openapi-maintain and local-ci-parity skills. Those need their own check, since the right replacement may not be verify.yml (some steps live in the shared dani-actions workflow).

How to verify

Docs-only; no code or behavior change.

  • make check-skills-docs and make check-stack-docs pass.
  • grep -n "ci\.yml" .claude/skills/doc-sync-patch/SKILL.md returns nothing.

🤖 Generated with Claude Code

The skill's description and anti-patterns still named .github/workflows/ci.yml,
which doesn't exist. Point at backend/Dockerfile instead, which check_stack_docs.py
now reads alongside go.mod and .nvmrc.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

Copy link
Copy Markdown
Contributor

PR guide

Auto-generated from changed paths. Prefer a why-first Summary in the PR description above (motivation, then what changed); use this comment for path-based checklist and suggested verify commands.

Touches: other

Diff: 1 file changed, 2 insertions(+), 2 deletions(-)

Suggested verify

  • N/A — tooling/docs only; confirm locally if anything user-facing changed

Checklist (applies to this PR)

  • No unintended secrets or local-only config committed

Reviewer focus

  • Scope looks tooling- or docs-only — confirm no hidden runtime impact

Commits

  • d9b7d3f docs(skills): drop stale ci.yml references from doc-sync-patch

Files by area

Area Files
other 1

CI

Required checks: Frontend (per-concern jobs) and Backend in verify. Coverage tables are posted separately when those jobs finish.


Template: pull_request_template.md

@danibsheehan
danibsheehan merged commit 9d8124f into main Oct 12, 2026
23 checks passed
@danibsheehan
danibsheehan deleted the docs/doc-sync-patch-ci-yml branch October 12, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant