Skip to content
This repository has been archived by the owner on Aug 17, 2020. It is now read-only.

Long execution time #4

Open
kubajakowski opened this issue Jun 4, 2020 · 3 comments
Open

Long execution time #4

kubajakowski opened this issue Jun 4, 2020 · 3 comments

Comments

@kubajakowski
Copy link

Hi,
converting our app's coverage files takes about 20 minutes - large codebase, a lot of tests. It doubles our standard PR's CI build execution time: build + test + sonar static analysis. Do you see any room for improvement in your lib? Maybe there's something on your roadmap? We'd be more than happy to help you with it. Or maybe there's something that we can do on our's project side to reduce conversion time?

Great job!
Cheers

@heinzl
Copy link
Collaborator

heinzl commented Jun 15, 2020

Hi, thanks for your feedback. I'm sorry but there are currently no plans for further improvements.

Or maybe there's something that we can do on our's project side to reduce conversion time?

If you use Swift Packages in your project you can try to avoid code coverage conversion at all by using swift test / xcrun and generate code coverage reports in llvm-cov. llvm-cov is directly supported by SonarQube, no conversion needed.

@guillaumeLeRoy
Copy link

Hi,

Converting our coverage files is also quite slow. Our project contains around 100k lines of code covered by almost 1K of Unit Tests. Xcode generates 2 xcresults, respectively of 2MB and 63MB which will be converted by your script.
We usually see a conversion time on Sonar of around 8min which we think is quite slow.
Do you think this conversion duration is normal for the size of our project?

@heinzl
Copy link
Collaborator

heinzl commented Aug 17, 2020

We are no longer using this script I described our new approach in the readme. The repository will be archived.

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

No branches or pull requests

3 participants