Skip to content

Dapper.AOT#417

Open
latop2604 wants to merge 2 commits intohangfire-postgres:masterfrom
latop2604:dapper-aot
Open

Dapper.AOT#417
latop2604 wants to merge 2 commits intohangfire-postgres:masterfrom
latop2604:dapper-aot

Conversation

@latop2604
Copy link

@latop2604 latop2604 commented Dec 12, 2025

Fix #416

Also:

  • merge some .Query().SingleOrDefault() into .QuerySingleOrDefault() as dapper throw the exact same exceptions as System.Linq.Enumerable.
  • Replace some .ToList() by .AsList().
  • Replace tuple-type by struct record as suggested in https://aot.dapperlib.dev/rules/DAP013
  • Convert JsonParameter to a static class. And add ::jsonb in sql script when needed.

@latop2604 latop2604 marked this pull request as draft December 12, 2025 19:49
@latop2604
Copy link
Author

Some tests are ko. Need more work on it.

@latop2604 latop2604 force-pushed the dapper-aot branch 6 times, most recently from 00335ed to 2346eac Compare December 12, 2025 20:56
@latop2604 latop2604 marked this pull request as ready for review December 12, 2025 21:02
@latop2604
Copy link
Author

@azygis
Copy link
Collaborator

azygis commented Dec 14, 2025

Looks good, thanks! I'll try to not forget about it tomorrow.

@latop2604
Copy link
Author

Any news?

@azygis
Copy link
Collaborator

azygis commented Feb 5, 2026

Sorry, "tomorrow" in my calendar looks to be something different than in everyone else's.

Will try to merge and publish this week.

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.

Feature: Move to DapperAOT?

2 participants