Skip to content

Add missing LINQ overloads for IRewritableQueryable#5

Merged
koenbeuk merged 14 commits intomainfrom
feat/extended-rewritable-queryable-coverage
Mar 28, 2026
Merged

Add missing LINQ overloads for IRewritableQueryable#5
koenbeuk merged 14 commits intomainfrom
feat/extended-rewritable-queryable-coverage

Conversation

@koenbeuk
Copy link
Copy Markdown
Collaborator

@koenbeuk koenbeuk commented Mar 27, 2026

Introduce additional LINQ overloads to enhance the functionality of IRewritableQueryable, improving usability and flexibility in query operations.

This PR got out of hand and it also fixes the feature gaps that were left over with our EFCore integration. I think I tackled all gaps with a notable exception to ExecuteUpdate support. We can address that as part of a separate PR

Copilot AI review requested due to automatic review settings March 27, 2026 23:16
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'ExpressiveSharp Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 44c2f4b Previous: 6808d2d Ratio
ExpressiveSharp.Benchmarks.ExpressionReplacerBenchmarks.Replace_Method 3305.939254760742 ns (± 1490.9362570026249) 2396.2288411458335 ns (± 597.6971388629727) 1.38
ExpressiveSharp.Benchmarks.GeneratorBenchmarks.RunGenerator_Incremental_ExpressiveChange(ExpressiveCount: 1000) 29223755.666666668 ns (± 939508.9888689092) 12475252.375 ns (± 349659.53509311064) 2.34

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 62 out of 62 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 103 out of 103 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@koenbeuk koenbeuk merged commit e82b015 into main Mar 28, 2026
3 checks passed
@koenbeuk koenbeuk deleted the feat/extended-rewritable-queryable-coverage branch March 28, 2026 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants