-
Notifications
You must be signed in to change notification settings - Fork 65
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
coverageLocations fails with more than once package #153
Comments
Hi @paambaati - here is a draft pr with update It's possible my syntax isn't correct. If you can share an example array that would be great. Thanks Thanks, |
I have the same problem. I believe the reason is described here: https://github.community/t5/GitHub-Actions/Can-action-inputs-be-arrays/td-p/33776
There are two workarounds:
I will try it and make a PR 🙂 |
Thanks @MartinNuc, appreciate your help. I was going to submit some doc updates as well as soon as I got it figured out. Were you able to get it to actually format_coverage, sum_coverage, and upload_coverage? If that works it's a step in the right direction for sure. The script methods I used on Circle CI seem to be broken on GH Actions now with the cc-test-reporter. In fact, cc-test-reporter itself doesn't seem to handle mono-repos well. Thanks again. |
Not yet. Still working on it. I had to convert also prefix into array. So files are being passed now as a multiline string (last segment after
Right now have a problem that |
@MartinNuc , I did a test and got through that part thanks to your suggestion. But, haven't had any luck yet getting format_coverage to work. In fact, I have a few threads on this issue, including a different approach scripting in the CI rather than using an Action. But, all roads for that have resulted in |
Finally got it working. My problem was actually with the instanbul coverage file. There were relative paths which didn't work well with monorepo like:
I had to set the
Which worked with formatter correctly. Here is the related issue: istanbuljs/istanbuljs#529 Solved by setting
After that it turned out that there is no need to use different prefix for each If you want you can try my version using:
I will post the PR to this repo. |
@MartinNuc Would be super-helpful if you can add these to the README's examples section as well. |
@paambaati Actually I did here https://github.com/paambaati/codeclimate-action/pull/163/files#diff-04c6e90faac2675aa89e2176d2eec7d8L44 I will add few more |
Thanks a ton @MartinNuc! This fix has now landed in |
Using coverageLocations fails to work. I've tried many different options and the only one that seems to work is running on a single package.
The text was updated successfully, but these errors were encountered: