-
Notifications
You must be signed in to change notification settings - Fork 2
Fix the Variable name for the test. #3
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
Conversation
Thank you for finding this issue! I'll make a new release soon. |
Hi :) This release does not work in my case, requiring Here's my debug output:
|
Once again, thank you for your contribution, but I jumped the gun earlier and should have done some more testing before making the release. I've rolled the release back, and I'm going to make a v2 release with the |
Oh hey @bytehead , you've run into the same issue as what I described above. v1.0.0 is again the latest - you can continue to trust |
Thank you @g105b! |
I keep getting this error:
Warning: include(/usr/local/bin/../phpmd/phpmd/src/bin/phpmd): Failed to open stream: No such file or directory in /usr/local/bin/phpmd on line 119
Warning: include(): Failed opening '/usr/local/bin/../phpmd/phpmd/src/bin/phpmd' for inclusion (include_path='.:/usr/local/lib/php') in /usr/local/bin/phpmd on line 119
when trying to use this action with a vendored phpmd.
I found the Variable name in the final test was $ACTION_PHPUNIT_PATH which I think should be $ACTION_PHPMD_PATH