-
Notifications
You must be signed in to change notification settings - Fork 29
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
Hotfix/high number fix #30
Conversation
Would using a UInt64 for execution counts be cleaner than parsing as a Double and converting back to Int? |
I didn't expect Codable to accept UInt64. |
Thanks for pointing that out. Apparently UInt64 is what Xcode is using for the executionCount. |
… for the hint, alexdeem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments inline and also below :)
It would be good if we could get the path URL to be relative like the code coverage output, this is needed for GitLab.
The check name could also be issueSummary.issueType
rather than a hash.
Not that this appears to show in GitLab anyhow.
Thank you for doing this!
No description provided.