Skip to content

ci: add pre-test checks with format, lint, and compile steps#11

Merged
rootandroo merged 2 commits into
mainfrom
ci-pretest-checks
May 29, 2026
Merged

ci: add pre-test checks with format, lint, and compile steps#11
rootandroo merged 2 commits into
mainfrom
ci-pretest-checks

Conversation

@rootandroo
Copy link
Copy Markdown

@rootandroo rootandroo commented May 10, 2026

Add pretest checks for types, style, formatting, and compilation.

Closes #81

@rootandroo rootandroo requested a review from a team as a code owner May 10, 2026 18:12
@rootandroo rootandroo force-pushed the ci-pretest-checks branch from a71455f to 2bc637e Compare May 10, 2026 18:17
arklian
arklian previously approved these changes May 14, 2026
Copy link
Copy Markdown

arklian commented May 14, 2026

You'll need to make PRs to format the backend and the frontend so that the checks pass the first time.

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
- Rename jobs: backend-ci → backend-pretest, frontend-ci → frontend-pretest
- Backend: add Spotless format check, Checkstyle, and compile steps
- Frontend: switch to pnpm, add typecheck, prettier, lint, and build steps
- Trigger on push to any branch and PRs targeting main
- Document trigger rules in workflow file
@rootandroo rootandroo force-pushed the ci-pretest-checks branch 2 times, most recently from 8c9f6c5 to c74b70f Compare May 24, 2026 12:11
@rootandroo rootandroo force-pushed the ci-pretest-checks branch from c74b70f to f83bbeb Compare May 24, 2026 12:14
@rootandroo rootandroo requested a review from a team as a code owner May 24, 2026 12:14
@sonarqubecloud
Copy link
Copy Markdown

@rootandroo rootandroo requested a review from arklian May 24, 2026 12:16
@rootandroo rootandroo merged commit f83bbeb into main May 29, 2026
7 checks passed
@rootandroo rootandroo deleted the ci-pretest-checks branch May 29, 2026 23:17
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.

2 participants