-
Notifications
You must be signed in to change notification settings - Fork 9
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 accessors for enzyme associated data #724
Conversation
@exaexa could you look at the accessors I added? I will fix the tests soon |
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 okay, some structural cleaning needed
dcbb253
to
13488c3
Compare
/format |
✔️ Auto-formatting triggered by this comment succeeded, commited as 94e7218 |
/format |
☀️ Auto-formatting triggered by this comment succeeded, but the code was already formatted correctly. |
5d39dfb
to
2db86bc
Compare
2db86bc
to
a0d2fe2
Compare
Codecov ReportBase: 89.61% // Head: 89.41% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## next #724 +/- ##
==========================================
- Coverage 89.61% 89.41% -0.20%
==========================================
Files 81 81
Lines 1916 1918 +2
==========================================
- Hits 1717 1715 -2
- Misses 199 203 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View 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.
LGTM, FFTM
No description provided.