Skip to content

✅ Improve SequenceSet#slice test coverage - #773

Merged
nevans merged 1 commit into
masterfrom
sequence_set/test-coverage
Oct 10, 2026
Merged

nevans merged 1 commit into
masterfrom
sequence_set/test-coverage

Conversation

@nevans

@nevans nevans commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

This uncovered several significant bugs and several lesser bugs (where it behaves differently from Array#slice). For this commit, those tests are all marked pending. They should be fixed ASAP.

@nevans nevans changed the title ✅ Improve SequenceSet slice test coverage ✅ Improve SequenceSet#slice test coverage Oct 3, 2026
@nevans
nevans force-pushed the sequence_set/test-coverage branch from 60f223a to 8891cbf Compare October 3, 2026 00:28
@nevans
nevans added this pull request to stack #775 October 3, 2026 00:32
@nevans
nevans force-pushed the sequence_set/test-coverage branch from 8891cbf to b230edc Compare October 3, 2026 13:42
This uncovered several significant bugs and several lesser bugs (where
it behaves differently from `Array#slice`).  For this commit, those
tests are all marked pending.  They should be fixed ASAP.
@nevans
nevans force-pushed the sequence_set/test-coverage branch from b230edc to 1b9d2ba Compare October 10, 2026 03:15
Base automatically changed from sequence_set/slice-rdoc to master October 10, 2026 03:25
@nevans
nevans merged commit 1b9d2ba into master Oct 10, 2026
34 checks passed
@nevans
nevans deleted the sequence_set/test-coverage branch October 10, 2026 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant