fix(ci): resolve Prettier formatting discrepancies in multiple files - #8924
Conversation
|
@nishtha-agarwal-211 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This PR resolves formatting issues that were causing CI failures, which is crucial for maintaining code quality. It's great to see the attention to detail in ensuring that all formatting checks pass. Please ensure that similar formatting checks are consistently applied in future contributions. Thank you for your efforts in keeping the codebase clean! |
There was a problem hiding this comment.
Pull request overview
This PR aims to resolve CI npm run format:check failures by applying Prettier formatting fixes across a couple of TS/TSX files and a documentation file.
Changes:
- Re-formatted TypeScript union types for readability/Prettier compliance.
- Re-formatted React/TSX code blocks in
InteractiveViewer.tsxto match Prettier output. - Updated
docs/customization.mdtable formatting (currently introduces a malformed table row).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| types/achievements.ts | Prettier-driven formatting of union type definition. |
| docs/customization.md | Attempts to adjust markdown table formatting, but currently breaks the table separator row. |
| components/InteractiveViewer.tsx | Prettier formatting of Array.from and map render blocks. |
| components/burnout/BurnoutRiskTable.tsx | Prettier-driven formatting of union type definition. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
🚨 Hey @nishtha-agarwal-211, the CI Pipeline is failing on this PR and it has been marked as Please fix the issues before this can be reviewed. Here's how: 1. Run checks locally before pushing: npm run format:check # Check Prettier formatting
npm run lint # Run ESLint
npm run typecheck # TypeScript type check
npm run test # Run unit tests (Vitest)
npm run build # Verify production build passes2. Auto-fix common issues: npm run format # Auto-fix formatting with Prettier
npm run lint -- --fix # Auto-fix lint errors where possible3. Check the full failure log here: Once you push a fix and the CI passes, the |
a295448 to
856db22
Compare
Description
Fixes #8915
Resolves CI code quality check (
npm run format:check) failure by fixing Prettier code formatting style discrepancies inBurnoutRiskTable.tsx,InteractiveViewer.tsx,customization.md, andachievements.ts.Pillar
Visual Preview
N/A (Formatting fix)
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.