Skip to content

Proj i06/ci baseline setup#79

Open
georges-nasrallah wants to merge 4 commits into
mainfrom
PROJ-I06/ci-baseline-setup
Open

Proj i06/ci baseline setup#79
georges-nasrallah wants to merge 4 commits into
mainfrom
PROJ-I06/ci-baseline-setup

Conversation

@georges-nasrallah

Copy link
Copy Markdown
Collaborator

Summary

Add baseline CI workflows for all services, dependabot for automated dependency updates and pre-commit config for local files hygiene checks. This is a baseline and needs to be modified.

Closes #7


Motivation & Context

No CI pipelines existed on the repo. PRs could be merged without any automaetd validation

Changes

  • Add separate PR CI workflows for auth (Maven + PostgreSQL service), genai (pip install), web-client (build only), and OpenAPI (Redocly lint)
  • Add dependabot watching all 5 package ecosystems (github actions, maven, pip, npm x2)
  • Add .pre-commit-config.yaml for file hygiene checks (trailing whitespaces, yaml/json validation and scret detection)

Steps for Testing

  1. Open any PR targetting main and verify the 4 CI checks appear and pass.
  2. Merge a dependency bump PR from dependabot once it raises one.
  3. Run pip install pre-commit && pre-commit install locally, then attempt to commit a file with trailing whitespace.

Checklist

  • [ x] I have self-reviewed the diff
  • [ x] Code is clean — no debug statements, commented-out code, or TODOs left behind
  • All CI checks pass (build · test · lint)
  • Tests added or updated to cover the changes (if needed)
  • New env vars added to .env.example
  • OpenAPI spec updated and codegen re-run (make -C api generate) if any endpoint changed
  • Screenshots or screen recordings attached for any UI changes

@georges-nasrallah georges-nasrallah requested a review from toukhi June 8, 2026 19:29
@georges-nasrallah georges-nasrallah self-assigned this Jun 8, 2026
@georges-nasrallah georges-nasrallah added setup infra Infrastructure & DevOps labels Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infrastructure & DevOps setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PROJ-I06] Add static analysis / linting step to CI pipeline

1 participant