-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: extend mature miRNA annotations #73
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.
Getting there. 3 more minor comments, I think after we can merge :)
If I am not wrong, we said that the extension would be for the user to define; provide this option on the config file has not yet been implemented. Do you still want the extension to be for the user to change or you rather leave it at 6 by the moment? |
Yes, it should indeed by up for the user, good that you remembered. Please add it to the config file; this PR is the right place to do this. Rest looks good now. |
While adding the |
Closes #65
In this PR the changes applied are:
gffutils
and thepytest
andcoverage