Skip to content

ci: add CI, PR template, CODEOWNERS (#5) - #13

Merged
andybaran merged 1 commit into
mainfrom
feat/5-ci-pr-template
Jul 16, 2026
Merged

ci: add CI, PR template, CODEOWNERS (#5)#13
andybaran merged 1 commit into
mainfrom
feat/5-ci-pr-template

Conversation

@andybaran

Copy link
Copy Markdown
Owner

Closes #5.

Closes #5.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@andybaran

Copy link
Copy Markdown
Owner Author

🔬 Code review (code-review agent)

Verdict: APPROVED — 0 Critical, 0 Important, 0 Minor.

Verified against Approved Plan v2, executed CI commands on the branch:

  • Scope clean & parallel-merge safe: only .github/CODEOWNERS, .github/pull_request_template.md, .github/workflows/ci.yml.
  • ShellCheck glob is the corrected post-rename path (skills/issue-driven-github-flow/scripts/*.sh tests/*.sh) — the fix that reversed v1 rejection; both targets exist, shellcheck exit 0.
  • Triggers pull_request + push to main; ubuntu-latest; pinned actions/checkout@v4 before shell steps; apt install with sudo ... update && -y.
  • Dependency-light: evals lint uses stdlib json only; no PyYAML, no blocking actionlint/yamllint. No swallowed failures (bare commands fail the job).
  • PR template checklist + CODEOWNERS * @andybaran valid. No secrets/stray files.

Human orchestrator performing final squash-merge.

@andybaran
andybaran marked this pull request as ready for review July 16, 2026 20:15
@andybaran
andybaran merged commit 589b6ce into main Jul 16, 2026
1 check 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.

Add CI (shellcheck + tests + evals lint), PR template with definition-of-done, and CODEOWNERS

1 participant