Skip to content

Feature/lab3#1050

Open
Mysteri0K1ng wants to merge 11 commits into
inno-devops-labs:mainfrom
Mysteri0K1ng:feature/lab3
Open

Feature/lab3#1050
Mysteri0K1ng wants to merge 11 commits into
inno-devops-labs:mainfrom
Mysteri0K1ng:feature/lab3

Conversation

@Mysteri0K1ng

Copy link
Copy Markdown

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>
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>
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