-
Notifications
You must be signed in to change notification settings - Fork 2
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
make test
command failed if we cannot build locally a PHAR version
#42
Comments
Once the PHAR version was compiled with BOX v4 (see #41 (comment)), the
|
Interesting. Maybe I never noticed it because for local development I have to use PHP 8.1 to ensure version compatibility. Thanks for your feedback, @llaville. It is very much appreciated. I will try to fix it soon. Anyway I hope it doesn't prevent you from using the tool. Any ideas would be very welcome. I will try to improve and add new data format if there is anything else. Thanks again for your attention to my project. |
You're right, it doesn't prevent me to use it, and even to run unit tests and codestyle (just after compile the new PHAR version with BOX v4) I think there is a great converter to add to your project. I'll propose to you later when dev will be over and tested with other reports. |
The related issue is solved #41 |
Related to issue #41, when we invoke
make test
command, we got following outputThe text was updated successfully, but these errors were encountered: