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

PHP 8.2 Support #5003

Closed
wants to merge 4 commits into from
Closed

Conversation

TAINCER
Copy link

@TAINCER TAINCER commented Feb 14, 2023

@cla-bot cla-bot bot added the cla/signed label Feb 14, 2023
@TAINCER TAINCER self-assigned this Feb 14, 2023
@TAINCER TAINCER force-pushed the PHP8.2-Support branch 4 times, most recently from ddadfae to 6b6ebd8 Compare February 15, 2023 13:09
@TAINCER TAINCER changed the base branch from 4918 to PHP8.2-vendor-update February 15, 2023 13:10
@TAINCER TAINCER changed the base branch from PHP8.2-vendor-update to move-vendor-packages-to-thirdparty April 13, 2023 09:30
@@ -193,7 +193,7 @@ public function visitColor($c)
{
if ($this->definingVariable !== false) {
// Make sure that all less tree colors do have a proper name
$c->name = $this->variableOrigin->name;
// $c->name = $this->variableOrigin->name;

Choose a reason for hiding this comment

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

it could be $c->name = $this->variableOrigin?->name; as variableOrigin could be null

@nilmerg nilmerg deleted the branch move-vendor-packages-to-thirdparty August 9, 2023 14:02
@nilmerg nilmerg closed this Aug 9, 2023
@nilmerg nilmerg deleted the PHP8.2-Support branch August 9, 2023 14:03
nilmerg added a commit that referenced this pull request Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants