Merged
Conversation
Initial open source release of EasyAF 3.1 as we head towards RTM
- `is` and `is not` fixes - First DotNetDocs generation
- LTree scaffolding fixes
…o we can open them in VS22/VS26
…ableCollection<T> (#1) ## Summary Introduces comprehensive bulk-operation extension methods for `Collection<T>` and `ObservableCollection<T>`, enabling efficient batch additions, insertions, removals, and replacements with optimized notification handling. Implements dotnet/runtime#18087 because Microsoft could never figure out how to get around to it. ## Key Changes - **New Extension Methods**: Added `AddRange()`, `InsertRange()`, `RemoveRange()`, and `ReplaceRange()` methods to both `Collection<T>` and `ObservableCollection<T>` - `Collection<T>` overloads use per-item virtual dispatch (preserving subclass validation) - `ObservableCollection<T>` overloads manipulate the inner `Items` list directly and raise single batched notifications - **Notification Modes**: Introduced `CollectionChangeNotificationMode` enum with two options: - `Batched` (default): Raises a single event with all affected items in `NewItems`/`OldItems` - `Reset`: Raises a single `Reset` action for WPF compatibility - **Reentrancy Protection**: Implemented reentrancy checks for `ObservableCollection<T>` operations to prevent invalid state during event handling with multiple subscribers - **Comprehensive Test Suite**: Added 573 lines of unit tests covering: - Correct item insertion/removal/replacement - Event notification behavior (batched vs. reset modes) - Property change notifications (`Count` and `Item[]`) - Boundary validation and error cases - Reentrancy scenarios - Overload resolution priority for `ObservableCollection<T>` ## Implementation Details - Uses C# 14 extension syntax for clean API surface - Leverages `UnsafeAccessor` for zero-overhead access to protected `Collection<T>.Items` property - Employs `OverloadResolutionPriority(1)` attribute to ensure `ObservableCollection<T>` overloads are preferred over `Collection<T>` overloads - Properly handles edge cases (empty ranges, null arguments, out-of-bounds indices) - Maintains consistency with standard .NET collection semantics
…at broke dependency chains.
- Add build-and-deploy.yml with NuGet Trusted Publishing - Add pr-validation.yml for PR build/test validation - Bump DotNetDocs.Sdk from 1.3.0 to 1.5.4 - Update docs.json navigation with new API reference sections - Regenerate API reference documentation Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Solution-level restore silently skips projects with analyzer ProjectReferences (Tests.Analyzers.EF6). Letting build handle restore for missed projects fixes this with minimal overhead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Tests in Tests.Business and Tests.Data.EF6 require SQL LocalDB
which is not available on GitHub Actions runners. Tagged with
TestCategory("RequiresDatabase") and excluded via --filter.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The OData TripPin demo service is intermittently flaky from CI runners. Retry up to 3 times with a random delay, and return Inconclusive instead of failing the build if all attempts fail. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code