-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add product names to report metadata. #61
Conversation
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.
Looks good, this is a great addition!
For your info; if you add an issue in the PR description using one of the closing keywords the issue will be linked and closed once the PR is merged.
I've now changed "Issue 60" to "Fixes 60" for you 🙂
Diagnostics.framework: Coverage: 77.12
Generated by 🚫 Danger Swift against a953948 |
@AvdLee OK, good to know! |
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.
Can you change the colons to match our code style?
Great stuff, merging in! |
Congratulations! 🎉 This was released as part of Release 1.6.0 🚀 |
Happy to help! 🙂 |
Only includes most recent iPhone models.
Fixes #60