-
Notifications
You must be signed in to change notification settings - Fork 81
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
[MISC] rename seqan3::{alignment => sam}_file_output* #2414
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/seqan/seqan3/DaFzC63zVAzRdS99gJEVdTLxBEdn |
Codecov Report
@@ Coverage Diff @@
## master #2414 +/- ##
=======================================
Coverage 98.21% 98.21%
=======================================
Files 267 267
Lines 10863 10863
=======================================
Hits 10669 10669
Misses 194 194
Continue to review full report at Codecov.
|
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.
💅
sam_file_output(std::filesystem::path, selected_field_ids const &) | ||
-> sam_file_output<selected_field_ids, | ||
typename sam_file_output<>::valid_formats, | ||
ref_info_not_given>; |
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.
sam_file_output(std::filesystem::path, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, | |
typename sam_file_output<>::valid_formats, | |
ref_info_not_given>; | |
sam_file_output(std::filesystem::path, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, typename sam_file_output<>::valid_formats, ref_info_not_given>; |
sam_file_output(stream_type &&, file_format const &, selected_field_ids const &) | ||
-> sam_file_output<selected_field_ids, | ||
type_list<file_format>, | ||
ref_info_not_given>; |
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.
sam_file_output(stream_type &&, file_format const &, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, | |
type_list<file_format>, | |
ref_info_not_given>; | |
sam_file_output(stream_type &&, file_format const &, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, type_list<file_format>, ref_info_not_given>; |
sam_file_output(stream_type &, file_format const &, selected_field_ids const &) | ||
-> sam_file_output<selected_field_ids, | ||
type_list<file_format>, | ||
ref_info_not_given>; |
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.
sam_file_output(stream_type &, file_format const &, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, | |
type_list<file_format>, | |
ref_info_not_given>; | |
sam_file_output(stream_type &, file_format const &, selected_field_ids const &) | |
-> sam_file_output<selected_field_ids, type_list<file_format>, ref_info_not_given>; |
sam_file_output_format file_format> | ||
sam_file_output(stream_type &&, file_format const &) | ||
-> sam_file_output<typename sam_file_output<>::selected_field_ids, | ||
type_list<file_format>, | ||
ref_info_not_given>; |
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.
same here
sam_file_output(stream_type &, file_format const &) | ||
-> sam_file_output<typename sam_file_output<>::selected_field_ids, | ||
type_list<file_format>, | ||
ref_info_not_given>; |
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.
same here
sam_file_output(stream_type &&, ref_ids_type &&, ref_lengths_type &&, file_format const &) | ||
-> sam_file_output<typename sam_file_output<>::selected_field_ids, | ||
type_list<file_format>, | ||
std::remove_reference_t<ref_ids_type>>; |
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.
sam_file_output(stream_type &&, ref_ids_type &&, ref_lengths_type &&, file_format const &) | |
-> sam_file_output<typename sam_file_output<>::selected_field_ids, | |
type_list<file_format>, | |
std::remove_reference_t<ref_ids_type>>; | |
sam_file_output(stream_type &&, ref_ids_type &&, ref_lengths_type &&, file_format const &) | |
-> sam_file_output<typename sam_file_output<>::selected_field_ids, | |
type_list<file_format>, | |
std::remove_reference_t<ref_ids_type>>; |
@Irallia ty good catch :) |
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.
perfect, thank you
Part of many small rename PRs, tackles seqan/product_backlog#290.