Skip to content

ci(lab3): add PR-gated pipeline with vet, test, and lint#922

Open
blacktree-lab wants to merge 19 commits into
inno-devops-labs:mainfrom
blacktree-lab:feature/lab3
Open

ci(lab3): add PR-gated pipeline with vet, test, and lint#922
blacktree-lab wants to merge 19 commits into
inno-devops-labs:mainfrom
blacktree-lab:feature/lab3

Conversation

@blacktree-lab

Copy link
Copy Markdown

Goal:

Add CI pipeline with vet, test, and lint as a required PR gate for QuickNotes.

Changes:

  • Added .github/workflows/ci.yml with vet, test, lint jobs
  • Added Go build matrix (1.23 + 1.24)
  • Added module caching, shallow clone, and path filter
  • Added submissions/lab3.md with design question answers and evidence

Testing:

  • CI runs green on all 5 matrix jobs (vet×2, test×2, lint)
  • Deliberately broke test — PR was blocked as expected
  • Fixed and confirmed green again
  • Path filter verified — README-only commit skipped CI
  • Branch protection requires all 3 checks to pass before merge

Checklist:

  • Title is a clear sentence (≤ 70 chars)
  • Commits are signed (git log --show-signature)
  • submissions/lab3.md updated

Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
Signed-off-by: DJ Bubu <djbubu28@yahoo.com>
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