Skip to content

feat: compatibility with symfony 6 and php 8#12

Open
lucassabreu wants to merge 5 commits into
yosymfony:masterfrom
lucassabreu:master
Open

feat: compatibility with symfony 6 and php 8#12
lucassabreu wants to merge 5 commits into
yosymfony:masterfrom
lucassabreu:master

Conversation

@lucassabreu

Copy link
Copy Markdown

Removed conflict with Symfony 6 and PHP 8.

I had to up the PHP version to 7.1 because of PHPUnit forcing void as return type on setUp and tearDown.

Also fixed PHPUnit tests failing using symfony/filesystem:2.7 (lowest).

@flexchar

flexchar commented Feb 16, 2022

Copy link
Copy Markdown

It would be amazing! Looking forward to using with Laravel 9 & PHP 8.1.

Required by https://github.com/spatie/phpunit-watcher.

Edit: now required by Laravel 11 & PHP 8.3. But it's overcome by using composer instal --global for development only related packages.

@robsontenorio

Copy link
Copy Markdown

Yes :)

@neklein

neklein commented Apr 18, 2022

Copy link
Copy Markdown

Any update on this pull request being merged? I need it to be compatible with symfony/finder v6.

Comment thread composer.json
"require": {
"php": ">=5.6",
"symfony/finder": "^2.7|^3.0|^4.0|^5.0"
"php": ">=7.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If we drop php 5.6 and 7.0 support, shouldn't 5.6 and 7.0 be removed from .travis.yml too?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree.
And more I think Travis-CI should be abandoned in favor of GH Actions.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yes, i forgot to remove it. will work on it

@tyteck

tyteck commented Oct 1, 2022

Copy link
Copy Markdown

How can I help to get this PR merged ?

@ArnaudLigny

Copy link
Copy Markdown
Contributor

@yosymfony 🙏

@frederikhs

Copy link
Copy Markdown

+1

@neklein

neklein commented Jan 10, 2023

Copy link
Copy Markdown

Happy New Year, @yosymfony - any chance we could get this package updated? Or are you still planning to support this package?

@xepozz

xepozz commented Mar 2, 2023

Copy link
Copy Markdown

@yosymfony Still waiting your move

@robsontenorio

Copy link
Copy Markdown

Is there any other compatible alternative?

@Kanti

Kanti commented Apr 25, 2023

Copy link
Copy Markdown

you can install composer req andersundsehr/resource-watcher:dev-master as a replacement for `yosymfony/resource-watcher``. Until this is resolved.

@cleberar

Copy link
Copy Markdown

Any solution for this problem?

@xepozz

xepozz commented Dec 29, 2023

Copy link
Copy Markdown

Fork ;)

@ArnaudLigny

Copy link
Copy Markdown
Contributor

For my part I use the fork of @lucassabreu:

"repositories": [
    {
      "type": "vcs",
      "url": "https://github.com/lucassabreu/resource-watcher"
    }
  ],

@kirkbushell

Copy link
Copy Markdown

Wow, this is still not resolved? Cmonnnnn....

@michaelw85

Copy link
Copy Markdown

At this point, I would like to add it's time to support the v7 packages and PHP 8.3. Has this project gone stale?

@lucassabreu

Copy link
Copy Markdown
Author

i am not using this anymore.

@michaelw85

Copy link
Copy Markdown

i am not using this anymore.

Did you switch to an alternative? I'm running into this issue because I'm updating dependencies for spaties/phpunit-watcher which has this package as a dep. I forked your branch and updated the v7 dep for symfony/finder.

pierresh pushed a commit to pierresh/phpunit-watcher that referenced this pull request May 20, 2024
Include yosymfony/resource-watcher directly in PHPUnit watcher so that it is easier to manage dependencies.

yosymfony/resource-watcher#12
pierresh pushed a commit to pierresh/phpunit-watcher that referenced this pull request May 20, 2024
Include yosymfony/resource-watcher directly in PHPUnit watcher so that it is easier to manage dependencies.

yosymfony/resource-watcher#12
pierresh pushed a commit to pierresh/phpunit-watcher that referenced this pull request May 22, 2024
Include yosymfony/resource-watcher directly in PHPUnit watcher so that it is easier to manage dependencies.

yosymfony/resource-watcher#12

Minimum PHP version 8.1

Require Symfony 6 or 7
freekmurze pushed a commit to spatie/phpunit-watcher that referenced this pull request May 22, 2024
* Bump symfony package version and update resource watcher

* Force new version

* Revert change

* Update test run with php 8.3 and re-add older phpunit versions

* Remove doc blocks in favor of annotations

* refactor: merge yosymfony/resource-watcher

Include yosymfony/resource-watcher directly in PHPUnit watcher so that it is easier to manage dependencies.

yosymfony/resource-watcher#12

Minimum PHP version 8.1

Require Symfony 6 or 7

---------

Co-authored-by: Michael Withagen <michael@lyfter.nl>
Co-authored-by: Michael Withagen <michael.withagen@live.com>
Co-authored-by: Pierre Clavequin <pierre.clavequin@valueapex.com>
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.