Skip to content

ci: gate PRs on ruff, eslint and frontend build - #1068

Merged
lfnovo merged 1 commit into
mainfrom
ci/lint-and-build-gates
Jul 11, 2026
Merged

ci: gate PRs on ruff, eslint and frontend build#1068
lfnovo merged 1 commit into
mainfrom
ci/lint-and-build-gates

Conversation

@lfnovo

@lfnovo lfnovo commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds three CI jobs to .github/workflows/test.yml: Backend Lint (ruff check .), Frontend Lint (npm run lint) and Frontend Build (npm run build)
  • These commands are documented as required in AGENTS.md but were not enforced on PRs until now
  • Jobs reuse the existing setup patterns (uv with cache, Node 22 with npm cache) and run in parallel with the test jobs
  • mypy is intentionally out of scope (separate PR later)

All three commands pass locally on current main.

Review in cubic

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You’re at about 92% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.

Re-trigger cubic

@lfnovo
lfnovo merged commit ac30f4d into main Jul 11, 2026
11 checks passed
@lfnovo
lfnovo deleted the ci/lint-and-build-gates branch September 2, 2026 22:03
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