Skip to content

ci(lab3): add PR gate#3

Open
Mysteri0K1ng wants to merge 9 commits into
mainfrom
feature/lab3
Open

ci(lab3): add PR gate#3
Mysteri0K1ng wants to merge 9 commits into
mainfrom
feature/lab3

Conversation

@Mysteri0K1ng

@Mysteri0K1ng Mysteri0K1ng commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Goal

Add a PR-gated GitHub Actions pipeline for QuickNotes and document Lab 3 CI/CD results.

Changes

  • Added .github/workflows/ci.yml with separate vet, test, and lint jobs
  • Pinned the runner version, action SHAs, Go versions, and golangci-lint version
  • Added Go dependency caching, Go 1.23/1.24 matrix jobs, and path filters
  • Documented green CI, failing CI, blocked merge evidence, timing measurements, and design answers in submissions/lab3.md
  • Added screenshots for CI evidence and branch protection behavior

Testing

Verified that the CI pipeline passes with vet, test, and lint jobs. I also intentionally broke a test to confirm that CI fails and the PR is blocked, then reverted the breaking commit and confirmed the pipeline returned to green.

Checklist

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

Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
@Mysteri0K1ng Mysteri0K1ng marked this pull request as ready for review June 15, 2026 20:46
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
This reverts commit eea2590.

Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.com>
Signed-off-by: Nikita Schankin <nikita.sshankin@gmail.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