Consider adding static analysis
BackEndTea opened this issue · 2 comments
- Algolia Client Version: N/A
- Language Version: N/A
Description
Consider using a static analysis tool (like PHPStan or Pslam), in order to catch basic errors, like the class not found issue introduced recently.
Currently there is no way to verify that code that is not covered by tests isn't broken. With static analysis all code would be automatically type checked, which should make it (nearly) impossible for these kind of errors to happen.
It shouldn't be too much work to set it up with real low strictness, which should catch any basic errors. At a later point it could be configured to be more strict, catching more specific errors.
If you need any help with setting this up, i'd be more than happy to help.
Hello, thanks for the report.
It's unfortunate that those kind bug are appearing. However there is good news, we are in the process of releasing a new major that will have better testing overall
Code is there: https://github.com/algolia/algoliasearch-client-php/tree/next
Source is there: https://github.com/algolia/api-clients-automation
Hey, thats great to see!
But even in a new major static analysis would still be beneficial to the overall quality of the project.
Plus it would also help better document types towards the user.
Especially since this is a new major, and adding/changing types could be a BC break, this seems like the perfect time to add those.