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.

Sign up for GitHub

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

Jump to bottom

Task: Update SortedSet<T>, SortedSet<T>.TreeSubSet implementation to match .NET 8 behavior #107

Closed
NightOwl888 opened this issue Sep 15, 2024 · 0 comments · Fixed by #110
Closed

Task: Update SortedSet<T>, SortedSet<T>.TreeSubSet implementation to match .NET 8 behavior #107

NightOwl888 opened this issue Sep 15, 2024 · 0 comments · Fixed by #110
Assignees
@NightOwl888
Labels
is:enhancement New feature or request pri:normal up for grabs This issue is open to be worked on by anyone
Milestone
2.1

Comments

@NightOwl888
Copy link
Owner

.NET 8 changes some collection behaviors. The test framework tests per interface and uses boolean switches to control tests for specific behaviors. We have configured the switches to make the tests pass based on the current behaviors rather than ensure they are in sync with .NET 8. So, the task is to go through the SortedSet<T> implementation and SortedSet<T>.TreeSubSet, bring in all changes between .NET 5 and .NET 8, and do the same with the SortedSet<T> tests, setting each of the boolean switches to match .NET 8 (as appropriate).
@NightOwl888 NightOwl888 added is:enhancement New feature or request pri:normal up for grabs This issue is open to be worked on by anyone labels Sep 15, 2024
@NightOwl888 NightOwl888 added this to the 2.1 milestone Sep 15, 2024
@NightOwl888 NightOwl888 self-assigned this Sep 15, 2024
NightOwl888 added a commit that referenced this issue Sep 16, 2024
@NightOwl888
SWEEP: Upgraded SortedSet<T> and SortedDictionary<TKey, TValue> to th…
10603cd 
…e .NET 8.0 implementation (fixes #107, fixes #108). Upgraded collection test framework and tests to account for new or differing behaviors in .NET 8.0 (excluding IList features). Added IReadOnlySet<T> to all set types.
@NightOwl888 NightOwl888 mentioned this issue Sep 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NightOwl888 NightOwl888

Labels
is:enhancement New feature or request pri:normal up for grabs This issue is open to be worked on by anyone
Projects
None yet
Milestone
2.1
Development

Successfully merging a pull request may close this issue.

Added net8.0 target and tests (Fixes #94) NightOwl888/J2N
1 participant
@NightOwl888