Repository navigation
chore: remove remaining AFX recruitment banner - #131
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe ChangesAdHire removal
Development dependency update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The retired recruitment banner is removed as intended. The dependency update has no established supported-Node or installation failure, so no actionable merge blocker is evidenced. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying cnpmweb with
|
| Latest commit: |
1dfc6e1
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://130bcee1.cnpmweb.pages.dev |
| Branch Preview URL: | https://codex-remove-package-hiring.cnpmweb.pages.dev |
There was a problem hiding this comment.
🟢 Approval recommended
The targeted cleanup is complete, and no obsolete references remain in src.
0 open findings
What changed in this PR
Removes the retired AFX recruitment banner and obsolete references.
Changes:
- Removes the package-page banner.
- Deletes the unused
AdHirecomponent. - Removes the obsolete homepage comment.
| File | Description |
|---|---|
src/pages/package/[...slug]/index.tsx |
Removes AdHire usage and import. |
src/pages/index.tsx |
Removes the obsolete comment. |
src/components/AdHire.tsx |
Deletes the retired banner component. |
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
🟢 Approval recommended
The retired banner is fully removed and the dependency changes are consistent with the existing test configuration.
0 open findings
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
Package detail pages still display the retired AFX recruitment banner after its removal from the homepage. The advertised department no longer exists.
Remove the remaining banner from package detail pages, delete the unused AdHire component and recruitment link, and clean up the obsolete homepage comment.
Also pin Vitest to 4.0.16 and declare the compatible Vite 7 peer dependency to unblock npm 10 dependency installation in CI and preview builds. The floating Vitest range now selects 4.1.11, whose peer graph reproducibly crashes npm 10 with
edgesOut.Validation:
src.