Enhance Monolith Height Scaling for High Contribution Profiles - #8329
Enhance Monolith Height Scaling for High Contribution Profiles#8329VanshikaSingh28 wants to merge 1 commit into
Conversation
|
@VanshikaSingh28 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 enhances the height scaling for high contribution profiles, which is a significant improvement for user experience. It is important to ensure that the changes align with previous decisions, such as fixing dead code in the calculateStreak function to maintain accurate streak statuses. Please review the checklist to ensure all requirements are met. Looking forward to your feedback! |
|
🚨 Hey @VanshikaSingh28, 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 |
b2e1d93 to
c769779
Compare
Aamod007
left a comment
There was a problem hiding this comment.
Monolith height scaling fix for high-contribution profiles looks correct — adjusting the SVG layout to accommodate taller contribution bars. CI is failing, please check the logs.
Labels: level:intermediate (4 files, SVG layout + tests), quality:clean, type:bug, mentor:Aamod007
Aamod007
left a comment
There was a problem hiding this comment.
Re-reviewing after noting the only CI failure is Vercel authorization (not code-related). All other checks (Format · Lint · Typecheck · Test, Production Build, CodeQL) pass. Approving — the Vercel deploy auth is a repo-level config issue, not a code problem.
…(#4146)
Description
Fixes #4146
Pillar
Visual Preview
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.