Skip to content

ci: add ci/cd pipeline and dockerfile (story 1.2)#2

Merged
vieiralucas merged 5 commits intomainfrom
feat/1-2-ci-cd-pipeline
Feb 11, 2026
Merged

ci: add ci/cd pipeline and dockerfile (story 1.2)#2
vieiralucas merged 5 commits intomainfrom
feat/1-2-ci-cd-pipeline

Conversation

@vieiralucas
Copy link
Copy Markdown
Member

@vieiralucas vieiralucas commented Feb 11, 2026

Summary

  • CI workflow with 4 parallel jobs (fmt, clippy, test, bench) triggered on PRs and pushes to main
  • Release workflow triggered on version tags with 4-target cross-platform build matrix
  • Multi-stage Dockerfile for container image published to ghcr.io
  • GitHub Release creation with binaries and SHA256 checksums

Test plan

  • CI workflow runs successfully on this PR
  • All 4 CI jobs (fmt, clippy, test, bench) pass
  • Release workflow syntax validates (will be tested on first tag push)
  • Dockerfile builds locally with docker build .

Summary by cubic

Adds a complete CI/CD pipeline with actions pinned to commit SHAs: CI runs fmt, clippy, tests, and benches on every pull request and on main; tagged releases build cross-platform binaries and publish a Docker image to GHCR. Satisfies Story 1.2 requirements for automated quality checks and release packaging.

  • New Features
    • CI on all PRs and main: fmt, clippy (-D warnings), nextest tests (placeholder test in fila-core), bench (non-blocking), installs protoc.
    • Release on v* tags: runs checks, builds fila and fila-server for linux amd64/arm64 and macOS amd64/arm64 (cross for linux arm64).
    • Packaging: GitHub Release with tarballs, LICENSE, and SHA256 checksums; Docker multi-stage image pushed to ghcr.io with semver tags; exposes 5555 with fila-server as entrypoint.

Written for commit 490ef1c. Summary will update on new commits.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 5 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name=".github/workflows/ci.yml">

<violation number="1" location=".github/workflows/ci.yml:17">
P2: Pin GitHub Actions to immutable commit SHAs instead of mutable tags to prevent unexpected or malicious updates in CI.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread .github/workflows/ci.yml Outdated
@vieiralucas vieiralucas force-pushed the feat/1-2-ci-cd-pipeline branch from d9f7556 to f4d3033 Compare February 11, 2026 12:03
Base automatically changed from feat/1-1-cargo-workspace-protobuf-definitions to main February 11, 2026 15:51
- ci workflow with fmt, clippy, test, bench jobs on PR and push to main
- release workflow triggered on version tags with 4-target build matrix
- multi-stage dockerfile for ghcr.io container image
- cross-compilation support for linux-arm64 via cross tool
revert --no-tests=pass flag in favor of an actual test
@vieiralucas vieiralucas force-pushed the feat/1-2-ci-cd-pipeline branch from f4d3033 to 490ef1c Compare February 11, 2026 15:52
@vieiralucas vieiralucas merged commit b3b6b50 into main Feb 11, 2026
4 checks passed
@vieiralucas vieiralucas deleted the feat/1-2-ci-cd-pipeline branch February 11, 2026 15:54
vieiralucas added a commit that referenced this pull request Feb 17, 2026
- Rename crate from fila-client to fila-sdk across workspace
- Add assertion after server startup poll to fail fast when server
  is unreachable (Cubic finding #1)
- Add comment explaining why None lease messages are filtered as
  expected keepalive frames (Cubic finding #2)
vieiralucas added a commit that referenced this pull request Feb 19, 2026
- Rename crate from fila-client to fila-sdk across workspace
- Add assertion after server startup poll to fail fast when server
  is unreachable (Cubic finding #1)
- Add comment explaining why None lease messages are filtered as
  expected keepalive frames (Cubic finding #2)
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