Skip to content

ci: add TypeScript check to PR workflow#17

Merged
rutefig merged 1 commit intomainfrom
rutefig/reg-635-add-typescript-check-to-ci-to-prevent-build-failures-on
Dec 18, 2025
Merged

ci: add TypeScript check to PR workflow#17
rutefig merged 1 commit intomainfrom
rutefig/reg-635-add-typescript-check-to-ci-to-prevent-build-failures-on

Conversation

@rutefig
Copy link
Copy Markdown
Member

@rutefig rutefig commented Dec 17, 2025

Runs yarn tsc --noEmit on all PRs targeting main to catch type errors before merge. This prevents build failures during Render deployment.

Summary by CodeRabbit

  • Chores
    • Added automated TypeScript type checking for pull requests to ensure code quality during development.

✏️ Tip: You can customize this high-level summary in your review settings.

Runs `yarn tsc --noEmit` on all PRs targeting main to catch type errors
before merge. This prevents build failures during Render deployment.
@rutefig rutefig requested a review from BornPsych December 17, 2025 19:10
@rutefig rutefig self-assigned this Dec 17, 2025
@linear
Copy link
Copy Markdown

linear Bot commented Dec 17, 2025

REG-635

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 17, 2025

Walkthrough

Introduces a new GitHub Actions CI workflow configured to automatically run TypeScript type checking on pull requests targeting the main branch. The workflow sets up Node.js 20, installs dependencies via yarn, and executes the TypeScript compiler in no-emit mode.

Changes

Cohort / File(s) Summary
GitHub Actions CI Workflow
.github/workflows/ci.yml
New workflow file for TypeScript type checking; triggers on pull requests to main branch; runs tsc --noEmit after environment setup and dependency installation

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single configuration file addition with standard GitHub Actions syntax
  • No logic changes or business logic to evaluate
  • Straightforward workflow structure following common CI patterns

Poem

🐰 A workflow we weave with types held tight,
TypeScript checks dance in the GitHub light,
No errors shall slip through our fuzzy sight,
Pull requests polished—oh, what a delight! ✨


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 459c3eb and 7447aaa.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml (1 hunks)

Note

🎁 Summarized by CodeRabbit Free

The PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@rutefig rutefig merged commit 945774f into main Dec 18, 2025
3 checks passed
@rutefig rutefig deleted the rutefig/reg-635-add-typescript-check-to-ci-to-prevent-build-failures-on branch December 18, 2025 08:16
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