-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
[DX] [Experimental] Add withPhpLevel() to raise PHP level one rule at a time #6261
Conversation
847bd5a
to
9a00924
Compare
5c000c9
to
b1e555a
Compare
b1e555a
to
4b99765
Compare
79fef6e
to
15cb9ea
Compare
a87fc6d
to
6c846ca
Compare
7f2c91b
to
56d7658
Compare
32a96fc
to
1cfdf1f
Compare
Merging this to test in the wild 👍 |
It seems cause error and different web view result with empty data after used new to ensure it has equal result. |
|
||
$this->isWithPhpLevelUsed = true; | ||
|
||
$phpVersion = ComposerJsonPhpVersionResolver::resolveFromCwdOrFail(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should allow read from withPhpVersion()
defined from Rector config so it can be tested on getrector.com, which PhpVersionProvider seems can be used, or add fallback on ComposerJsonPhpVersionResolver
to use predefined from withPhpversion() config when exists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created experiment PR at #6384
No description provided.