-
Notifications
You must be signed in to change notification settings - Fork 473
Linter fix quast #7152
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
base: main
Are you sure you want to change the base?
Linter fix quast #7152
Conversation
and thereby also use id and ext for mate pairs
feca53b
to
6454a06
Compare
Feeling tempted to revert #5149. The case of individual assembly should be covered by using nested collections. |
Co-authored-by: Saim Momin <64724322+SaimMomin12@users.noreply.github.com>
Since 242cc0a might be controversial I would like to get more feedback on this. |
puh, yeah controversial, but I'm ok with it ... |
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.
@bernt-matthias I think we have no objections to this as such
Lets ping the micogalaxy community |
I keep looking at this PR. It will for sure simplify things but I am also afraid it will break things and might be complicated for users (but probably just a matter of training 😅) |
@bernt-matthias if you have @bebatut blessing ... I guess you can proceed :) |
I planned to add a bit of documentation. Just found no time yet |
FOR CONTRIBUTOR: