Skip to content

ci(lab3): add PR-gated pipeline with caching, matrix, optimizations#1033

Open
infernaltiger wants to merge 15 commits into
inno-devops-labs:mainfrom
infernaltiger:feature/lab3
Open

ci(lab3): add PR-gated pipeline with caching, matrix, optimizations#1033
infernaltiger wants to merge 15 commits into
inno-devops-labs:mainfrom
infernaltiger:feature/lab3

Conversation

@infernaltiger

Copy link
Copy Markdown

Goal

Implement a PR-gated CI pipeline for QuickNotes with caching, build matrix, path filtering, and performance optimizations.

Changes

  • Added .github/workflows/ci.yml with three independent jobs: vet, test, lint
  • Set permissions: contents: read (least privilege principle)
  • Pinned runner to ubuntu-24.04 and golangci-lint to v2.5.0
  • Enabled Go module caching via actions/setup-go keyed by app/go.sum
  • Added build matrix for Go 1.23 and 1.24 with fail-fast: false
  • Added path filtering to skip pipeline on docs-only changes
  • Added bonus optimizations: manual golangci-lint cache, concurrency group, checkout tuning
  • Configured branch protection on fork's main requiring CI to pass
  • Added submissions/lab3.md with timing tables, design question answers, and bottleneck analysis

Testing

Checklist

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

Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@gmail.com>
Signed-off-by: Levak <levakov2003@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