Skip to content

ci: add GitHub Actions for Next.js build and Python lint#51

Merged
William-Hill merged 1 commit intomainfrom
ci/github-actions
Feb 18, 2026
Merged

ci: add GitHub Actions for Next.js build and Python lint#51
William-Hill merged 1 commit intomainfrom
ci/github-actions

Conversation

@William-Hill
Copy link
Collaborator

Summary

  • Next.js build (.github/workflows/nextjs-build.yml): runs npm run lint and npm run build on any PR touching codebenders-dashboard/. Triggers on PRs to main and rebranding/bishop-state.
  • Python lint (.github/workflows/python-lint.yml): runs ruff check on ai_model/ and operations/ on any PR touching those paths or requirements.txt. Uses ruff (no heavy ML deps needed).

Why targeting main

Workflow files must exist on the target branch to run on PRs. Merging this to main first ensures all subsequent PRs (including the rebranding epic) get checked automatically.

Test plan

  • Open a PR that touches codebenders-dashboard/ and confirm the Next.js build check runs
  • Open a PR that touches ai_model/ and confirm the Python lint check runs
  • Confirm a PR with a TypeScript error fails the build check

- Auto-fix 69 F541/F401 errors (bare f-strings, unused imports)
- Add noqa: E402 to intentional post-sys.path imports in complete_ml_pipeline.py and complete_ml_pipeline_csv_only.py
@William-Hill William-Hill merged commit 7087437 into main Feb 18, 2026
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