-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Scheme name not shown on danger error message #19
Comments
@joanromano How do you run the plugin multiple times for multiple schemes? Running |
Hey @macdoum1 what I do is generate the markdown manually. You can do so by accessing your generated |
Awesome, makes sense! Thanks!
- Michael MacDougall
… On Jun 27, 2018, at 7:36 PM, Joan Romano ***@***.***> wrote:
Hey @macdoum1 what I do is generate the markdown manually. You can do so by accessing your generated Report's targets property and then markdown_value in each of them. You can find all that information in Xcov models
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@joanromano I'm trying to accomplish exactly what you did. Can you give more specific example of how you managed to generate code coverage for different test targets, without Danger overwriting it? |
@Vadimkomis sorry I don't have that code with me anymore, but as I said before you should be able to access all the generated Reports and manually tweak them |
@joanromano are you talking about this function in the Xcov modals: |
Hi, thanks for this plugin - I am trying to integrate it within our ci pipeline and so far the integration it's been pretty smooth.
We have a workspace composed of many subprojects/schemes, and I run the xcov plugin multiple times within our Dangerfile. This gives me different coverage reports per different schemes which is pretty handy. The only issue I am facing is that, if I get errors on minimum coverage not met in more than one scheme, the message does not show the name of that particular scheme:
Would it be possible to change this to show the name of that scheme as well? I'd be more than happy to submit a PR if that's the case.
The text was updated successfully, but these errors were encountered: