search

codeclimate / php-test-reporter

DEPRECATED PHP Test Reporter
Other
65 stars 31 forks source link

PHP error via travis.ci #107

Closed thomasplevy closed 7 years ago

thomasplevy commented 7 years ago

I've just started trying to set up code climate and I think I finally got everything working but builds run via travis are failing with the following error:

PHP Parse error:  syntax error, unexpected 'foreach' (T_FOREACH) in /home/travis/build/gocodebox/lifterlms/vendor/codeclimate/php-test-reporter/composer/bin/test-reporter on line 13

Parse error: syntax error, unexpected 'foreach' (T_FOREACH) in /home/travis/build/gocodebox/lifterlms/vendor/codeclimate/php-test-reporter/composer/bin/test-reporter on line 13

I tried to fix this locally but I can't get it to successfully run via command line either, hitting namespace errors when I try to execute the file at ./vendor/bin/test-reporter.

I tried to get a .phar to try it that way but I can't find .phars in any of the releases here

Thanks,

ABaldwinHunter commented 7 years ago

@thomasplevy thanks for the report! hoping that pr fixes this issue.

ABaldwinHunter commented 7 years ago

@thomasplevy curious about what version of php you're using.

Also, merged that PR. can you try updating to use latest master and see if that fixes the issue you were seeing?

thomasplevy commented 7 years ago

@ABaldwinHunter that appears to have fixed it thanks, was running tests on 5.6, 7, 7.1

While you're paying attention to me I have an unrelated issue I can't seem to find a repo token for my repo (it's open source) does code coverage not apply to the OS repos via code climate?

Thanks for the speedy update!

ABaldwinHunter commented 7 years ago

@thomasplevy np! yep code coverage totally applies to OS repos!

You should be able to find the token under test coverage tab in your repo settings.

For ref here're some potentially helpful docs in order of anticipated utility:

With OSS you can also enable PR integration and use our browser extension for chrome if you're a chrome user.

Feel free to ping me with any other queries. Let me know if you have any other trouble finding the repo token.

thomasplevy commented 7 years ago

@ABaldwinHunter ah it seems I had linked my account but not the organization.

Thanks for your help!!!