-
Notifications
You must be signed in to change notification settings - Fork 13
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
ENH: add action to collate genomes to GenomeData[DNASequences] #99
ENH: add action to collate genomes to GenomeData[DNASequences] #99
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #99 +/- ##
==========================================
+ Coverage 98.32% 98.39% +0.07%
==========================================
Files 39 39
Lines 2631 2748 +117
==========================================
+ Hits 2587 2704 +117
Misses 44 44 ☔ View full report in Codecov by Sentry. |
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.
Hey @DorielaGrabocka, can you please rebase this branch on top of main? The changes here should not have anything to do with the changes from the other PR, right? It's a completely independent action.
217e2bb
to
d895f92
Compare
d98f67c
to
c72d5e9
Compare
bc188d7
to
1bc46f6
Compare
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! Before we merge, could you please just update the description in plugin_setup? (see below) - thanks! 🙏
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.
Hey @DorielaGrabocka, sorry to un-approve but I just had another look and realized that your tests actually need a bit more work - please see the comment below.
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! 🏅
Closes #97