Skip to content
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

Appveyor - OpenCover > Coveralls #23

Closed
d4nj0nes opened this issue Mar 26, 2015 · 5 comments
Closed

Appveyor - OpenCover > Coveralls #23

d4nj0nes opened this issue Mar 26, 2015 · 5 comments

Comments

@d4nj0nes
Copy link

Hi - I have asked a similar question over at the Open Cover issues page as I am not sure where the problem lies and I was looking for guidance on best practice on this.
The scenario is that we run OpenCover on appveyor on two assemblies with a filter two include only two specific namespaces. Still the output file is over 100mb.
The generated file is then sent with Coveralls.Net to our coveralls.io repo.
The whole process takes about 20 minutes - partly I imagine due to the generation of the output xml partly the sending the large file (+ sources?) to coveralls via coveralls.net.

  • what are the options if any on sending less information (I have asked Open Cover for guidance on reducing the size of that file)

  • we are using the fork at https://github.com/jdeering/coveralls.net from nuget version 1.25-beta. We updated to 1.3.2 from 1.2.5-beta which gave an error about not finding a value for SourceDigest in the CoverageFile . To get around that we added the option for --full-sources , but then ultimately reverted back to the 1.2.5-beta. Is this a known issue? Again perhaps a question for Open Cover about the output file formatting

    Any help would be appreciated thanks.

@jdeering
Copy link
Member

You'll definitely want the more up-to-date versions as we added support for sending the source digest to coveralls instead of the full source. I will work on seeing A) what the issue was for getting the SourceDigest value and B) a way to speed up the hashing.

If the majority of the time is OpenCover, you'll definitely want to get with them to slim it down, but I will work on trying to keep Coveralls.net from being the bottleneck as much as possible.

@d4nj0nes
Copy link
Author

Thank for your reply. I have had a few pointers from them on this so will try those. We would definitely like to be using the latest version as soon as possible.

@jdeering
Copy link
Member

Version 1.3.4 has been released. It should resolve the issue with digests that you all were having. If not, it should at least provide more details so we can better narrow down your specific issue.

@d4nj0nes
Copy link
Author

Thanks that's great - I'll give it a try now...

@d4nj0nes
Copy link
Author

That worked perfectly and no performance issues. Thanks for the quick turn around!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants