Skip to content

Add PR Quality Gates#60

Merged
pstaylor-patrick merged 5 commits intomainfrom
fix/lint
Nov 6, 2025
Merged

Add PR Quality Gates#60
pstaylor-patrick merged 5 commits intomainfrom
fix/lint

Conversation

@pstaylor-patrick
Copy link
Collaborator

@pstaylor-patrick pstaylor-patrick commented Nov 4, 2025

🚧 This PR is Part of a Series

👋 TL;DR

This PR adds CI workflow and code formatting tools to improve code quality enforcement and automated checks.

🔎 Details

This update implements automated quality gates and code formatting:

  • GitHub Actions workflow for pull request quality gates including format check, linting, type checking, build, and tests
  • Prettier integration for consistent code formatting with format check and format scripts
  • Configuration updates including prettierignore file to exclude generated files
  • Environment documentation updated to remove deprecated npx auth secret command reference
  • Test scripts added for unit and e2e testing
  • Package dependencies updated to include Prettier
  • Package lock regeneration to ensure dependency consistency

✅ How to Test

  1. CI Workflow: Verify GitHub Actions run successfully on PR with all quality gates passing
  2. Code Formatting: Run npm run format:check to ensure code follows Prettier standards
  3. Format Scripts: Test npm run format applies formatting correctly without errors
  4. Build Process: Confirm npm run build completes successfully
  5. Testing: Run npm test and npm run test:e2e to ensure all tests pass
  6. Environment Setup: Verify .env.example updates are appropriate for new contributors

🥜 GIF

lack-of-hustle

@netlify
Copy link

netlify bot commented Nov 4, 2025

Deploy Preview for f3-the-codex-demo ready!

Name Link
🔨 Latest commit dce0115
🔍 Latest deploy log https://app.netlify.com/projects/f3-the-codex-demo/deploys/690c11638723820008c88d5d
😎 Deploy Preview https://deploy-preview-60--f3-the-codex-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pstaylor-patrick pstaylor-patrick force-pushed the fix/lint branch 2 times, most recently from babc40d to 21120dc Compare November 4, 2025 20:53
@pstaylor-patrick pstaylor-patrick changed the title fix lint errs + add pr quality gates add pr quality gates Nov 4, 2025
@pstaylor-patrick pstaylor-patrick changed the title add pr quality gates Add PR Quality Gates Nov 4, 2025
This was referenced Nov 4, 2025
- Add GitHub Actions workflow for pull request quality gates including format check, linting, type checking, build, and tests
- Add Prettier for code formatting with format check and format scripts
- Add prettierignore file to exclude generated files from formatting
- Update .env.example to remove deprecated npx auth secret command reference
- Add test scripts for unit and e2e testing
- Update package dependencies to include Prettier

These changes improve code quality enforcement through automated CI checks and consistent code formatting across the project.
Updated tsconfig.tsbuildinfo file to reflect changes in TypeScript
library dependencies and project file structure. This ensures proper
incremental compilation and type checking by including the latest
type definitions and project references.
@pstaylor-patrick pstaylor-patrick merged commit 5814764 into main Nov 6, 2025
8 of 9 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.

1 participant