Skip to content

Add CI workflow for PR validation#4

Merged
athmangude merged 3 commits into
mainfrom
copilot/add-lint-and-build-workflow
Feb 12, 2026
Merged

Add CI workflow for PR validation#4
athmangude merged 3 commits into
mainfrom
copilot/add-lint-and-build-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 12, 2026

PRs currently have no automated checks. This adds CI validation to ensure code quality before merge.

Changes

package.json

  • Added lint script using tsc --noEmit for type checking without build artifacts

.github/workflows/pr-checks.yml

  • Triggers on pull_request events (opened, synchronize, reopened)
  • Runs lint, build, and test sequentially on Node.js 18
  • Uses npm ci for reproducible dependency installation
  • Sets permissions: contents: read (least privilege)

The workflow blocks PRs with type errors, build failures, or test failures.

Original prompt

This section details on the original issue you should resolve

<issue_title>Lint and build PRs must work before PR is merged</issue_title>
<issue_description>Github workflows must run lint and build scripts for all PRs every time they are updated</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 12, 2026 18:16
Co-authored-by: athmangude <2655082+athmangude@users.noreply.github.com>
Co-authored-by: athmangude <2655082+athmangude@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds CI validation to ensure pull requests pass TypeScript type-checking, build, and Jest tests before merge, aligning with the repo’s goal of catching issues early in PRs.

Changes:

  • Added a lint npm script that runs tsc --noEmit for TypeScript type-checking.
  • Introduced a GitHub Actions workflow to run lint, build, and test on pull_request events.
  • Configured the workflow to use Node.js 18 with npm dependency caching and minimal contents: read permissions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Adds a lint script for TypeScript type-checking without emitting build artifacts.
.github/workflows/pr-checks.yml Adds PR-triggered CI job to install deps and run lint/build/tests with secure permissions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI changed the title [WIP] Add lint and build workflows for PR validation Add CI workflow for PR validation Feb 12, 2026
Copilot AI requested a review from athmangude February 12, 2026 18:19
@athmangude athmangude marked this pull request as ready for review February 12, 2026 18:28
@athmangude athmangude merged commit 42427ba into main Feb 12, 2026
7 checks passed
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.

Lint and build jobs must work before PR is merged

3 participants