-
Notifications
You must be signed in to change notification settings - Fork 360
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
Test fails due to different suggestion text, when nearly all extensions are installed #2041
Comments
Using class names that won't have conflicts in different namespaces is one way to fix this. Additionally, because the global namespace has many more class names (of each allowed string length for a substitute), Phan gave up on checking for selections of certain lengths because of |
@remicollet - do you have a link to the steps you used to set up that PHP binary? Also, tests/run_all_tests may reveal more issues |
Alternate suggestions may have been generated depending on what PHP modules were installed with the PHP binary used to run the unit tests. - Rename some classes to avoid this ambiguity - Increase the suggestion check limit so that test results will include all suggestions - Add `%S` where suggestions might go (a wildcard with 0 or more characters) For phan#2041
Alternate suggestions may have been generated depending on what PHP modules were installed with the PHP binary used to run the unit tests. - Rename some classes to avoid this ambiguity - Increase the suggestion check limit so that test results will include all suggestions - Add `%S` where suggestions might go (a wildcard with 0 or more characters) For phan#2041
It is my dev computer Mostly Fedora 27 (for now) and "dnf install php*" |
The reported warnings were fixed. Followup issues can be filed if any other issues are seen. |
Some seems related to cassandra 1.3.2, phalcon 3.4.1 , swoole 4.2.1 being enabled during the test suite
The text was updated successfully, but these errors were encountered: