-
-
Notifications
You must be signed in to change notification settings - Fork 346
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
Skip files with short <?= PHP tag as leads to invalid changes #6068
Conversation
063b699
to
bdd783c
Compare
bdd783c
to
58957d5
Compare
d043a71
to
34ab183
Compare
34ab183
to
642ca88
Compare
…handle broken symlinks and notContains()
Let's see how downgrade goes ... Edit: all good ✔️ |
@staabm @paulbalandan are using windows iirc, could you give some insight? Thank you. |
The release note should include the recommendation for migrating to long php tag, Also probably better, add warning that list of files will not be processed, so user can know that the code should be migrated to long php tag first when running it ... |
That would be helpful indeed. Could you add this? |
I will try |
Windows filter files symlink fixed at PR: |
What exactly needs to be tested/fixed in windows? |
@staabm on windows already fixed at PR: |
I created PR to report files start with |
Currently, PHP allows to use
<?=
tags intead of typical<?php ...
: https://www.php.net/manual/en/language.basic-syntax.phptags.phpYet, such tags would lead to breaking changes like:
We currently skip this on importing files, but we should not process such files at all, to avoid breaking code.
Use php-cs-fixer rule https://cs.symfony.com/doc/rules/php_tag/echo_tag_syntax.html to migrate your short tags. Then use Rector safely 👍