"Fix comment stripping regex removing URL content inside strings"#1112
Conversation
|
@namrarafique93-del is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
Playwright smoke tests are failing on this PR. Check the CI logs, fix failing tests, then push an update. Cannot merge while tests are red. |
|
@Priyanshu-byte-coder Done please review again !!! |
c4663c2
into
Priyanshu-byte-coder:main
|
🎉 Merged! Thanks for contributing to DevTrack. If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers. Keep an eye on open issues for your next contribution! |
…iyanshu-byte-coder#1112) * "Fix comment stripping regex removing URL content inside strings" * fix: remove duplicate playwright test * fix: resolve playwright footer selector issue
…iyanshu-byte-coder#1112) * "Fix comment stripping regex removing URL content inside strings" * fix: remove duplicate playwright test * fix: resolve playwright footer selector issue
Summary
Fixes an edge case where the single-line comment stripping regex could incorrectly remove URL content inside string literals.
Closes #1110
Type of Change
Changes Made
How to Test
Steps for the reviewer to verify this works:
const url = "https://example.com";
node scripts/check-missing-deps.js
Screenshots (if UI change)
N/A
Checklist