Add .NET 10 / EF Core 10 / AutoMapper 16 support#87
Add .NET 10 / EF Core 10 / AutoMapper 16 support#87jbogard merged 2 commits intoAutoMapper:masterfrom
Conversation
- Add net10.0 target framework - Update AutoMapper to 16.x, AutoMapper.Collection to 13.x, ExpressionMapping to 10.x - Update EF Core per TFM: 8.0.16 (net8.0), 9.0.7 (net9.0), 10.0.5 (net10.0) - Update test dependencies: FluentAssertions 8.9.0, xunit 2.9.3, Test SDK 18.3.0 - Update tooling: Roslynator 4.15.0, MinVer 7.0.0 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
windows-latest may not have .NET 8/9/10 pre-installed, so explicitly set up all required SDKs for multi-targeting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@dotnet-policy-service agree company="egritosgroup" |
|
@jbogard who can check and accept this? |
|
@xrkolovos can you request a PR review from Jimmy? |
|
@lbargaoanu @Nicot3 can you merge this pr? thank you |
|
@xrkolovos I've just have a quick look without testing or building but, it is possible to add more versions? For example, AutoMapper.Extensions on version 12 is compatible with .NET 10, and with this change is not going to be possible to use this version anymore. This will force update to some people that maybe don't need to update his dependencies. I'm not 100% sure anyways, need to investigate a bit more or get a second opinion 😄 |
|
merge so people can update and avoid the new security vulnerability then solve your dependencies issues in another pr @Nicot3 |
do this with a seperated pull request @Nicot3 it will be a realease soon? |
|
@xrkolovos I don't have permissions on the repo, I can just approve 😅
|
|
@jbogard can you please approve this changes and make a release? |
|
another reason to let ai replace automapper for https://github.com/riok/mapperly ?? |
I did a quick read to Mapperly and doesn't have to seem the advantage of Dependency Injection. |
Auto-generated DTOs! Having DTOs change automatically because an underlying domain model changed feels risky. It effectively couples your API surface directly to internal implementation details, which can lead to:
In practice, this means a simple internal refactor could ripple out and break clients without any conscious decision to do so. For internal-only mappings this might be fine, but for externally exposed APIs, I’d strongly prefer DTOs to remain explicitly defined and intentionally evolved, rather than implicitly generated. To me, the convenience here doesn’t outweigh the loss of control over something as critical as the API contract. |
|
mapperly is just an easy fast way to do manual mapping using source generation and for the most part it doesn't need di |
|
@aarondglover I totally agree your point, for public contracts like APIs this needs to be consistent and should not change on each build because it can leads for broken contracts. But as you mention, for internal mappings it can reduce a lot of work. For now I didn't it on this way as on most of my projects I'm still using Automapper, but if I should change my library I will totally go with that one because the number of features it has are bigger an useful.
|
|
Just an FYI folks, I'm not the maintainer of ANY of the repositories here. I'm only a maintainer on the main AutoMapper repo. I have no idea what's going on in any of these repos. |
Summary
Test plan