Skip to content

Conversation

@kyle69
Copy link
Contributor

@kyle69 kyle69 commented Jan 2, 2026

Support for patching Dictionary<TKey, TValue> properties using indexer syntax.

@jeremydmiller
Copy link
Member

That's about the nastiest, hardest area of the Marten codebase to go into. Patching code looks good, but that introduced a regression in the LINQ support we'll need to figure out before this can go out,.

@kyle69 kyle69 force-pushed the feat/patching-dictionary branch from beea165 to f9a1cf7 Compare January 2, 2026 21:35
@kyle69
Copy link
Contributor Author

kyle69 commented Jan 2, 2026

That's about the nastiest, hardest area of the Marten codebase to go into. Patching code looks good, but that introduced a regression in the LINQ support we'll need to figure out before this can go out,.

I pushed a fix :)

Support for patching Dictionary<TKey, TValue> properties using indexer syntax.
@kyle69 kyle69 force-pushed the feat/patching-dictionary branch from f9a1cf7 to 0ea401a Compare January 2, 2026 21:54
@kyle69
Copy link
Contributor Author

kyle69 commented Jan 14, 2026

@jeremydmiller I'm following up on this—were you able to look into the build error issue?
Thank you

@mysticmind
Copy link
Member

Hey @kyle69 I will take a look and keep you posted sometime this week.

@jeremydmiller jeremydmiller added this to the 8.18.0 milestone Jan 14, 2026
@jeremydmiller
Copy link
Member

@kyle69 @babuannamalai I'm pulling this in locally to check out what's up w/ LINQ and get this in

@jeremydmiller
Copy link
Member

Looks good locally

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.

3 participants