Remove commented-out AdSense script block#357
Remove commented-out AdSense script block#357DSingh0304 wants to merge 2 commits intokeploy:mainfrom
Conversation
Signed-off-by: Deep <deep@example.com>
|
Kilo Code Review could not run — your account is out of credits. Add credits or switch to a free model to enable reviews on this change. |
There was a problem hiding this comment.
Pull request overview
Removes a dead, commented-out Google AdSense <Script /> JSX block from the custom Next.js Document to prevent lint/compile failures during CI runs.
Changes:
- Deleted the commented-out AdSense
<Script />block frompages/_document.tsx<Head>.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hey @DSingh0304 👋 — thanks so much for contributing to the project, really appreciate it! Your PR looks great and has been marked for merging. Here's a quick note from the reviewer:
We'll get this merged in soon. Keep the great work coming! 🚀 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Deep Shekhar Singh <deepshekhar0306@gmail.com>
Thank you sir!! |
Description
PR #249 commented out the Google AdSense
<Script />block inpages/_document.tsxbut left the JSX inside the block comment. ESLint'sreact/jsx-no-undefrule parses JSX component names even inside block comments, which caused a compile error and broke the Playwright e2e CI pipeline.This PR removes the dead commented-out block entirely to fix the CI failure.
Changes
<Script />block from<Head>inpages/_document.tsxType of Change
Testing
The Playwright e2e CI was failing with:
Removing the commented block resolves the compile error and allows the web server to start successfully.
Environment and Dependencies
Checklist