-
Notifications
You must be signed in to change notification settings - Fork 3
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
Gpl 745 sample and plate UUID #158
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #158 +/- ##
===========================================
+ Coverage 90.23% 90.73% +0.50%
===========================================
Files 13 13
Lines 1219 1285 +66
Branches 142 148 +6
===========================================
+ Hits 1100 1166 +66
Misses 91 91
Partials 28 28
Continue to review full report at Codecov.
|
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 great, some minor comments 👍
@harrietc52 just replied to some of your comments. Let me know what you think |
@harrietc52 do you mind approving so I don't have to force merge without approval? |
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.
@Chris-Friend sorry missed your comments yesterday. Agree with your tests comments, me and Andrew were discussing a wider refactor of crawler for this reason. Merge away.
Closes #156
Changes proposed in this pull request: