-
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
noodles-htsget based refactor #89
Comments
Closed via #122, and small changes in various other pull requests. |
@mmalenic Are we really reusing functionality from noodles-htsget? I cannot see it even being imported on https://github.com/umccr/htsget-rs/blob/main/htsget-search/Cargo.toml :/ IIRC, when looking at the Let me know if I'm wrong, if I'm not, reopen please ;) |
My bad! For some reason I had completely forgotten about that side of noodles. You're right, some of the structs look like they could be used from |
Revisit noodles newest changes on noodles-htsget crate and refactor our
htsget-search
accordingly: https://github.com/zaeleus/noodles/commits/master/noodles-htsgetThe text was updated successfully, but these errors were encountered: