refactor(it): move SeqToSeq2 from map.go to seq.go#898
Conversation
Adds doc page for SeqToSeq2.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #898 +/- ##
==========================================
+ Coverage 92.58% 92.60% +0.01%
==========================================
Files 32 32
Lines 5082 5082
==========================================
+ Hits 4705 4706 +1
+ Misses 274 273 -1
Partials 103 103
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR relocates the SeqToSeq2 iterator helper to the more appropriate it/seq.go module, and updates associated tests/examples and documentation accordingly.
Changes:
- Moved
SeqToSeq2implementation fromit/map.gotoit/seq.goand clarified its doc comment (index starts at 0 and increments by 1). - Moved
SeqToSeq2test and example from map-focused test files to sequence-focused test files (addingmapsimports where needed). - Added a new docs page for
SeqToSeq2underdocs/data.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| it/seq.go | Adds SeqToSeq2 implementation and doc comment in the sequence helpers file. |
| it/seq_test.go | Adds TestSeqToSeq2 and imports maps for maps.Collect. |
| it/seq_example_test.go | Adds ExampleSeqToSeq2 and imports maps. |
| it/map.go | Removes SeqToSeq2 from the map helpers file. |
| it/map_test.go | Removes TestSeqToSeq2 from map tests. |
| it/map_example_test.go | Removes ExampleSeqToSeq2 from map examples. |
| docs/data/it-seqtoseq2.md | Adds a new documentation page for SeqToSeq2. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Summary
SeqToSeq2fromit/map.gotoit/seq.go(correct file — it operates on sequences, not maps)map_test.go/map_example_test.gotoseq_test.go/seq_example_test.gomapsimport toseq_test.goandseq_example_test.godocs/data/it-seqtoseq2.md