Skip to content

[deprecation] Add explicit warning to withPhpPolyfill() deprecated method + raise error on ->withPhpSets() miss-use on PHP 7.4- projects#6790

Merged
TomasVotruba merged 3 commits intomainfrom
tv-newlines-output-warning
Mar 20, 2025
Merged

[deprecation] Add explicit warning to withPhpPolyfill() deprecated method + raise error on ->withPhpSets() miss-use on PHP 7.4- projects#6790
TomasVotruba merged 3 commits intomainfrom
tv-newlines-output-warning

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Mar 20, 2025

These methods have been deprecated for couple months now, one raising warnings. Time to increase warning to exception to remove last couple of miss-use in configs 👍

Ref rectorphp/rector#8957

@TomasVotruba TomasVotruba changed the title tv newlines output warning [deprecation] Add explicit warning to withPhpPolyfill() deprecated method + raise error on ->withPhpSets() miss-use on older projects Mar 20, 2025
@TomasVotruba TomasVotruba changed the title [deprecation] Add explicit warning to withPhpPolyfill() deprecated method + raise error on ->withPhpSets() miss-use on older projects [deprecation] Add explicit warning to withPhpPolyfill() deprecated method + raise error on ->withPhpSets() miss-use on PHP 7.4- projects Mar 20, 2025
@TomasVotruba TomasVotruba force-pushed the tv-newlines-output-warning branch from 3e1c31f to 010730c Compare March 20, 2025 09:56
@TomasVotruba TomasVotruba merged commit b337d37 into main Mar 20, 2025
44 checks passed
@TomasVotruba TomasVotruba deleted the tv-newlines-output-warning branch March 20, 2025 09:58
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 7, 2025

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant