Add CI/CD workflow with linting, formatting, and build checks#4
Merged
Add CI/CD workflow with linting, formatting, and build checks#4
Conversation
Contributor
BenGWeeks
commented
Jan 4, 2026
BenGWeeks
commented
- Add ESLint and Prettier as dev dependencies
- Add npm scripts for lint, lint:fix, format, and format:check
- Create .eslintrc.js with TypeScript and React best practices
- Create CI workflow (.github/workflows/ci.yml) that runs on PRs and pushes:
- ESLint linting checks
- Prettier format verification
- TypeScript type checking
- Build verification with artifact upload
- Add ESLint and Prettier as dev dependencies - Add npm scripts for lint, lint:fix, format, and format:check - Create .eslintrc.js with TypeScript and React best practices - Create CI workflow (.github/workflows/ci.yml) that runs on PRs and pushes: - ESLint linting checks - Prettier format verification - TypeScript type checking - Build verification with artifact upload
- Add Linting section with npm run lint and lint:fix commands - Add Formatting section with npm run format and format:check commands - Add CI/CD section describing GitHub Actions checks
There was a problem hiding this comment.
Pull request overview
This pull request establishes a comprehensive CI/CD pipeline for the project by adding automated code quality checks through GitHub Actions, along with ESLint and Prettier tooling for local development.
- Adds ESLint and Prettier configurations with TypeScript and React support
- Introduces npm scripts for linting and formatting operations
- Creates a GitHub Actions CI workflow that runs on pushes and pull requests to verify code quality, formatting, type safety, and build success
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| package.json | Adds new npm scripts (lint, lint:fix, format, format:check) and installs ESLint/Prettier related dev dependencies |
| .eslintrc.js | Creates ESLint configuration with TypeScript, React, and React Hooks plugins, extends Prettier config to avoid conflicts |
| .github/workflows/ci.yml | Implements CI workflow with four parallel jobs: linting, format checking, type checking, and building with artifact upload |
| README.md | Documents new linting and formatting commands, and describes the CI/CD pipeline in a dedicated section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Add typecheck job to build dependencies to ensure type checking passes before build - Document CI=false rationale (React treats warnings as errors in CI mode) - Include backend/ directory in lint and format scripts for consistent coverage - Fix overly broad *.config.js ignore pattern to only target root-level configs
- Remove conflicting ESLint dependencies (use react-scripts built-in) - Simplify .eslintrc.js to extend react-app config - Format all source files with Prettier - Fix tsconfig.json to exclude backend from TypeScript compilation - Fix setupTests.test.tsx to use global expect with jest-dom matchers - Update package-lock.json with resolved dependencies
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.