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
[NETBEANS-5599] PHP 8.1 Support: Final class constants part #3331
[NETBEANS-5599] PHP 8.1 Support: Final class constants part #3331
Conversation
https://issues.apache.org/jira/browse/NETBEANS-5599 - Fix the grammar file - Add `FinalModifierHintError` and `PHP81UnhandledError` - Add unit tests
* under the License. | ||
*/ | ||
|
||
class FinalMethod { |
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.
Add missing unit tests for final methods.
@tmysik I've started supporting PHP 8.1. Could you please review this when you have time? If there is no problem, let's merge it. I've created the new branch( |
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.
Looks good to me, thanks for all the tests.
@tmysik Thank you as always :) |
@junichi11 You are the one doing the great work, not me ;) |
https://issues.apache.org/jira/browse/NETBEANS-5599
RFC: https://wiki.php.net/rfc/final_class_const
FinalModifierHintError
andPHP81UnhandledError
Screenshots
PHP Version:
with PHP 8.1
with PHP 8.0
Errors for the final modifier