Skip to content
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

[FEATURE] seqan3::structure_record #2380

Merged
merged 3 commits into from
Feb 17, 2021

Conversation

marehr
Copy link
Member

@marehr marehr commented Feb 15, 2021

Part of seqan/product_backlog#289

This implements

@marehr marehr requested a review from joergi-w February 15, 2021 23:40
@codecov
Copy link

codecov bot commented Feb 15, 2021

Codecov Report

Merging #2380 (1495d4b) into master (a1d5d4d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2380   +/-   ##
=======================================
  Coverage   98.21%   98.21%           
=======================================
  Files         265      266    +1     
  Lines       10806    10821   +15     
=======================================
+ Hits        10613    10628   +15     
  Misses        193      193           
Impacted Files Coverage Δ
include/seqan3/io/structure_file/input.hpp 100.00% <ø> (ø)
include/seqan3/io/structure_file/record.hpp 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1d5d4d...bc86c46. Read the comment docs.

Copy link
Member

@joergi-w joergi-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you!

@joergi-w joergi-w requested review from a team and eseiler and removed request for a team February 16, 2021 10:31
@eseiler eseiler merged commit 87bb1e1 into seqan:master Feb 17, 2021
@marehr marehr deleted the io_explicit_record_type_nr2 branch February 17, 2021 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants