Skip to content

Commit

Permalink
Allow PHP 8 and xdebug-handler 2.0
Browse files Browse the repository at this point in the history
  • Loading branch information
weirdan committed Apr 9, 2021
1 parent e7a6dcc commit 7b1cf3d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
6 changes: 3 additions & 3 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,13 @@
"performance"
],
"require": {
"composer/xdebug-handler": "^1.3",
"php": "~5.3 || ~7.0"
"composer/xdebug-handler": "^1.3 || ^2.0",
"php": "~5.3 || ~7.0 || ~8.0"
},
"require-dev": {
"phpunit/phpunit": "^5.0 || ^6.0 || ^7.0 || ^8.0 || ^9.0",
"ext-xdebug": "*",
"php": "~5.6 || ~7.0",
"php": "~5.6 || ~7.0 || ~8.0",
"squizlabs/php_codesniffer": "^3.3",
"php-parallel-lint/php-parallel-lint": "^1.3"
},
Expand Down
1 change: 0 additions & 1 deletion src/Restarter.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

class Restarter extends XdebugHandler
{
/** @param string $command */
protected function restart($command)
{
assert(null !== $this->tmpIni);
Expand Down

3 comments on commit 7b1cf3d

@johnstevenson
Copy link

Choose a reason for hiding this comment

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

Can't see a need for ^1.3 || ^2.0 as the php requirements have not changed so ^2.0 will always be installed (unless I'm missing something).

@weirdan
Copy link
Owner Author

Choose a reason for hiding this comment

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

This should allow installing RWX alongside with packages that depend on 1.3.

@johnstevenson
Copy link

Choose a reason for hiding this comment

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

Ah, good point - thanks.

Please sign in to comment.