[MISC] deprecate seqan3::views::move

[marehr]

resolve seqan/product_backlog#328

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/seqan/seqan3/AHKAzFwKpS6B5UjhVxqEWs2a3sjs
✅ Preview: https://seqan3-git-fork-marehr-deprecatemove-seqan.vercel.app

[codecov]

Codecov Report

Merging #2563 (4bc7c3e) into master (922b838) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 4bc7c3e differs from pull request most recent head 72acbe7. Consider uploading reports for the commit 72acbe7 to get more accurate results

@@            Coverage Diff             @@
##           master    #2563      +/-   ##
==========================================
- Coverage   98.23%   98.23%   -0.01%     
==========================================
  Files         269      269              
  Lines       10549    10539      -10     
==========================================
- Hits        10363    10353      -10     
  Misses        186      186              

Impacted Files	Coverage Δ
include/seqan3/range/views/move.hpp	100.00% <ø> (ø)
.../seqan3/range/container/concatenated_sequences.hpp	96.84% <0.00%> (-0.07%)	⬇️
include/seqan3/io/record.hpp	100.00% <0.00%> (ø)
include/seqan3/alphabet/cigar/cigar.hpp	91.30% <0.00%> (ø)
include/seqan3/range/views/as_const.hpp	100.00% <0.00%> (ø)
include/seqan3/io/sam_file/format_bam.hpp	96.29% <0.00%> (ø)
include/seqan3/io/sam_file/format_sam.hpp	97.44% <0.00%> (ø)
include/seqan3/io/sequence_file/input.hpp	100.00% <0.00%> (ø)
include/seqan3/alphabet/aminoacid/aa20.hpp	100.00% <0.00%> (ø)
include/seqan3/alphabet/aminoacid/aa27.hpp	100.00% <0.00%> (ø)
... and 33 more

---

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 a8afe8f...72acbe7. Read the comment docs.

[SGSSGene]

👍

[eseiler]

Looks good, but what about

seqan3/doc/tutorial/sequence_file/sequence_file_solution3.cpp

Line 90 in 8b4cfe0

| seqan3::views::move // mark ID to be moved out of record

Snippets build because it is a snippet with a workaround that is always active on GCC

[marehr]

Looks good, but what about

seqan3/doc/tutorial/sequence_file/sequence_file_solution3.cpp

Line 90 in 8b4cfe0

| seqan3::views::move // mark ID to be moved out of record

Snippets build because it is a snippet with a workaround that is always active on GCC

Ty good catch

[eseiler]

I removed the SEQAN3_WORKAROUND_GCC_96070 workaround from sequence_file_solution3.cpp