Skip to content

feat: compatibility with symfony 6 and php 8 #12

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lucassabreu
Copy link

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
Copy link

flexchar commented Feb 16, 2022

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

Yes :)

@neklein
Copy link

neklein commented Apr 18, 2022

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

@@ -11,12 +16,12 @@
}
],
"require": {
"php": ">=5.6",
"symfony/finder": "^2.7|^3.0|^4.0|^5.0"
"php": ">=7.1",

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
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
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
Copy link

tyteck commented Oct 1, 2022

How can I help to get this PR merged ?

@ArnaudLigny
Copy link
Contributor

@yosymfony 🙏

@frederikhs
Copy link

+1

@neklein
Copy link

neklein commented Jan 10, 2023

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

@xepozz
Copy link

xepozz commented Mar 2, 2023

@yosymfony Still waiting your move

@robsontenorio
Copy link

Is there any other compatible alternative?

@Kanti
Copy link

Kanti commented Apr 25, 2023

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

@cleberar
Copy link

Any solution for this problem?

@xepozz
Copy link

xepozz commented Dec 29, 2023

Fork ;)

@ArnaudLigny
Copy link
Contributor

For my part I use the fork of @lucassabreu:

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

@kirkbushell
Copy link

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

@michaelw85
Copy link

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
Author

i am not using this anymore.

@michaelw85
Copy link

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.