Skip to content

fix: fail CI on lint/tsc errors, add Docker builds and security scanning - #17

Merged
Senthil455 merged 1 commit into
mainfrom
fix/ci-fail-on-errors
Jun 15, 2026
Merged

fix: fail CI on lint/tsc errors, add Docker builds and security scanning#17
Senthil455 merged 1 commit into
mainfrom
fix/ci-fail-on-errors

Conversation

@Senthil455

Copy link
Copy Markdown
Owner

Changes

Fixed three CI issues: lint/tsc errors silently passing, no Docker builds, no security scanning.

What changed:

  • Lint (line 31): Removed || echo "Lint warning" — lint errors now fail the build
  • TypeCheck (line 33): Removed || true — TypeScript errors now fail the build
  • Added npm test to Frontend CI
  • Added npm audit to frontend and Node service jobs (high+ severity, continues on error)
  • Added gosec security scanning to Go services job
  • Added bandit security scanning to Python services job
  • Added docker-build job — builds all 9 service images using Docker Buildx with GitHub Actions cache. Pushes to ghcr.io on push events, only builds on PRs.
  • Added security-scan job — filesystem Trivy scan for HIGH/CRITICAL vulnerabilities, uploads SARIF results to GitHub Security tab

@Senthil455
Senthil455 merged commit 6d7978b into main Jun 15, 2026
1 of 19 checks passed

@Senthil455 Senthil455 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

gh: Not Found (HTTP 404)
{"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}

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