Release 3.0.0-rc.1 readiness - #3
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2315d89b3b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| builder.Append(" = new "); | ||
| builder.Append(leaf.ReadConverter.ConverterTypeName); | ||
| builder.AppendLine("();"); |
There was a problem hiding this comment.
Reutilize a instância configurada do conversor gerado
Quando ConvertFromDatabaseUsing<TConverter, TDatabase>() usa um conversor com estado por instância ou inicialização dependente do ambiente, este campo cria uma segunda instância global, diferente daquela criada e armazenada no PropertyMap. Como o materializador é selecionado comparando apenas os tipos do conversor, consultas geradas passam a produzir resultados diferentes do fallback e runtimes isolados compartilham indevidamente o mesmo conversor estático; registre o materializador com a instância capturada pela configuração ou não gere esse caminho sem poder preservá-la.
AGENTS.md reference: AGENTS.md:L176-L180
Useful? React with 👍 / 👎.
Summary
This PR prepares the forked
3.0.0-rc.1line againstmaster, collecting the changes accumulated since the last published upstream version/tag (v2.0.0). It is intentionally opened as a draft so the release evidence and remaining publication decision can be reviewed before merge.What changed since the last version
MappedGridReader, unbuffered reads, and async streaming.FluentMapRuntime, compatibility bridge behavior for the static API, and a newDapper.FluentMap.DependencyInjectionpackage.DFM001-DFM015diagnostic surface.CHANGELOG.md,COMPATIBILITY.md,MIGRATION.md,SUPPORT.md, and SDD evidence for Etapas 7-12 and the RC cycle.Release and packaging notes
3.0.0-rc.1; default local package output is guarded as development prerelease output rather than accidentally producing2.0.0or stable3.0.0.lib/:Dapper.FluentMap,Dapper.FluentMap.Dommel, andDapper.FluentMap.DependencyInjection.ProjectReferencefor core, Dommel, Dependency Injection, analyzer, generator, and trimmed publish scenarios.Known limitations / release status
3.0.0-rc.1publication was blocked because.sdd/release-3.0.0-rc.1/PUBLISH-AUTHORIZATION.mdwas absent.v3.0.0-rc.1tag was created, and no GitHub Release was created as part of the recorded RC.6 outcome.Validation recorded in the branch
dotnet restore ./Dapper.FluentMap.slndotnet build ./Dapper.FluentMap.sln --configuration Release --no-restoredotnet test ./Dapper.FluentMap.sln --configuration Release --no-builddotnet pack ./Dapper.FluentMap.sln --configuration Release --no-build -p:Version=3.0.0-rc.1eng/validate-release-artifacts.ps1eng/consumer-smoke/run-consumer-smoke.ps1dotnet list ./Dapper.FluentMap.sln package --vulnerable --include-transitiveReview focus
3.0.0line.