Skip to content
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

[BUGFIX] Revert broken support for multiple comments #741

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

oliverklee
Copy link
Contributor

This reverts e4c66f6 (#672), which broke comment parsing in strict mode.

We'll need to re-implement support for multiple comments later in a way that does not break the existing comment parsing.

This is the v8.x backport of #740.

This reverts e4c66f6 (#672), which broke comment parsing in
strict mode.

We'll need to re-implement support for multiple comments later
in a way that does not break the existing comment parsing.

This is the v8.x backport of #740.
@oliverklee oliverklee added this to the 8.6.1 or 8.7.0 milestone Oct 24, 2024
@oliverklee oliverklee self-assigned this Oct 24, 2024
@JakeQZ JakeQZ merged commit 81582a7 into v8.x Oct 24, 2024
25 checks passed
@JakeQZ JakeQZ deleted the backport/comments-bugfix branch October 24, 2024 16:10
westonruter added a commit to westonruter/PHP-CSS-Parser that referenced this pull request Nov 4, 2024
…into fix/malformed-identifier

* 'v8.x' of https://github.com/MyIntervals/PHP-CSS-Parser: (46 commits)
  [CLEANUP] Autoformat the code (MyIntervals#749)
  [TASK] Prepare the 8.7.0 release (MyIntervals#742)
  [BUGFIX] Fix another implictly nullable parameter (MyIntervals#751)
  [CLEANUP] Autoformat the changelog (MyIntervals#750)
  [TASK] Configure PHPUnit to be more strict (MyIntervals#744)
  [BUGFIX] Avoid implicit nullable parameters (MyIntervals#746)
  [BUGFIX] Revert broken support for multiple comments (MyIntervals#741)
  [TASK] Add some more tests for parsing comments (MyIntervals#739)
  [TASK] Use fixed PHPUnit versions (MyIntervals#726)
  [TASK] Update PHPUnit (MyIntervals#725)
  [TASK] State since which version code is `@internal`/`@deprecated` (MyIntervals#723)
  [TASK] Deprecate the expansion of shorthand properties (MyIntervals#719)
  [TASK] Mark parsing-internal classes and methods as `@internal` (MyIntervals#711)
  [TASK] Deprecate `Parser::setCharset()` and `Parser::getCharset()` (MyIntervals#703)
  [FEATURE] Run the tests on CI with PHP 8.4 as well (MyIntervals#701)
  [TASK] Run the tests with PHP versions up to 8.3 (MyIntervals#697)
  [TASK] Also allow higher versions of PHPUnit (MyIntervals#682)
  [BUGFIX] Fix type errors in PHP strict mode (MyIntervals#695)
  [TASK] Block installations on unsupported higher PHP versions (MyIntervals#692)
  [BUGFIX] Avoid using `setUp()` in testcases (MyIntervals#683)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants