Skip to content

fix: comma in regex value of magic comments #10100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 21, 2025

Conversation

LingyuCoder
Copy link
Contributor

Summary

fix #7356

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@LingyuCoder LingyuCoder requested a review from Copilot April 21, 2025 06:07
@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Apr 21, 2025
Copy link

netlify bot commented Apr 21, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 9761081
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6805e2d76112d60008ce8c59

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request fixes an issue with handling commas in regular expression values of magic comments. The changes include updating the regex pattern in WEBPACK_MAGIC_COMMENT_REGEXP to correctly handle escaped characters and commas, and adding a comprehensive test suite for various magic comment value types.

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Apr 21, 2025
@LingyuCoder LingyuCoder force-pushed the fix/magic-comments-regex-comma branch from 44767a2 to 9761081 Compare April 21, 2025 06:16
Copy link

codspeed-hq bot commented Apr 21, 2025

CodSpeed Performance Report

Merging #10100 will not alter performance

Comparing fix/magic-comments-regex-comma (9761081) with main (971c977)

Summary

✅ 11 untouched benchmarks

@LingyuCoder LingyuCoder requested a review from ahabhgk April 21, 2025 06:47
@LingyuCoder LingyuCoder enabled auto-merge (squash) April 21, 2025 06:47
@LingyuCoder LingyuCoder merged commit eeb3421 into main Apr 21, 2025
27 checks passed
@LingyuCoder LingyuCoder deleted the fix/magic-comments-regex-comma branch April 21, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Incorrect parsing of webpackInclude
2 participants