Conversation
|
@nyandika is attempting to deploy a commit to the React Developers Kenya Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Working on the failing tests. Will send an update |
|
@orama254 Could you authorize this PR to deploy to vercel to get the pre-release view |
|
@greptile |
Greptile SummaryThis PR updates dependencies to address Next.js vulnerabilities, bumping Next.js from 12.3.1 to 16.1.6, React from 18.1.0 to 19.2.4, and modernizing the build toolchain. The Key changes:
Critical issue:
Confidence Score: 2/5
|
| Filename | Overview |
|---|---|
| package.json | Major version bumps for Next.js (12.3.1 → 16.1.6), React (18.1.0 → 19.2.4), and build tools; lint script simplified |
| tailwind.config.js | Duplicate extend object at lines 6 and 16 - second will override first, losing initial animation/fontFamily config |
| eslint.config.mjs | New ESLint flat config for Next.js with proper global ignores |
| next.config.js | Updated image config to use remotePatterns (replacing deprecated domains), removed eslint.dirs |
| tsconfig.json | Changed JSX from preserve to react-jsx for React 19 compatibility |
| .github/workflows/validate.yml | Added prod branch trigger, updated Node.js 14.x → 24.x, updated actions to v6 |
| .github/workflows/e2e.yml | Added prod branch trigger, updated Node.js 18.x → 24.x, updated actions to v5/v6 |
| src/components/HeroHeader/HeroHeader.tsx | Added requestAnimationFrame for initial width setting; removed unused React import; cleanup added to useEffect |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Dependency Updates] --> B[Next.js 12.3.1 → 16.1.6]
A --> C[React 18.1.0 → 19.2.4]
A --> D[Tailwind 3.1.2 → 4.2.0]
B --> E[New ESLint Flat Config]
B --> F[Update next.config.js APIs]
C --> G[Change TSConfig jsx to react-jsx]
C --> H[Remove React imports]
D --> I[Update PostCSS plugin]
D --> J[Update tailwind.config.js]
E --> K[Run Prettier/Linting]
F --> K
G --> K
H --> K
I --> K
J --> K
K --> L[38 Files Changed]
J -.-> M[⚠️ Duplicate extend objects]
style M fill:#ff6b6b,stroke:#c92a2a,color:#fff
style A fill:#4dabf7,stroke:#1971c2,color:#fff
style L fill:#51cf66,stroke:#2f9e44,color:#fff
Last reviewed commit: 41570df
|
@orama254 fixed issues brought about by merge from prod |
|
@greptile |
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Changes proposed
There have been recent vulnerabilities in the nextjs space. Taking this chance to bump up all package versions
Also added prod branch to github actions. This necessisated a rerun of prettier and linting hence the other numerous file changes
Check List (Check all the applicable boxes)
Screenshots
N/A
Note to reviewers
N/A