Skip to content

[codex] Stabilize generator CI and snapshot baselines#270

Merged
HavenDV merged 5 commits intomainfrom
codex/ci-shared-generator-fixes
Apr 11, 2026
Merged

[codex] Stabilize generator CI and snapshot baselines#270
HavenDV merged 5 commits intomainfrom
codex/ci-shared-generator-fixes

Conversation

@HavenDV
Copy link
Copy Markdown
Collaborator

@HavenDV HavenDV commented Apr 9, 2026

Summary

  • emit shared options support per generated namespace in the source generator
  • avoid emitting Newtonsoft property converter attributes for converters that are only generated for System.Text.Json
  • refresh snapshot baselines and stale HTTP JSON assertions to match current generated output

Validation

  • dotnet build src/libs/AutoSDK.SourceGenerators/AutoSDK.SourceGenerators.csproj /p:TreatWarningsAsErrors=true
  • dotnet test src/tests/AutoSDK.SnapshotTests/AutoSDK.SnapshotTests.csproj --filter "FullyQualifiedName~ParallelGeneration"
  • dotnet test src/tests/AutoSDK.SnapshotTests/AutoSDK.SnapshotTests.csproj --filter "FullyQualifiedName~SdkGenerator"
  • dotnet test src/tests/AutoSDK.UnitTests/AutoSDK.UnitTests.csproj
  • dotnet test src/tests/AutoSDK.IntegrationTests.Cli/AutoSDK.IntegrationTests.Cli.csproj --filter "FullyQualifiedNameGenerate_WithRequestAndClientOptions_EmitsOptionsSupportAndBuilds|FullyQualifiedNameGenerate_WithSystemNetHttpJsonFlag_UsesHttpJsonHelpers_AndBuilds|FullyQualifiedNameGenerate_WithOpenApi32QueryStringCookieAndAdditionalOperations_BuildsAndEmitsSupport|FullyQualifiedNameGenerate_WithOpenApi32SequentialMediaTypesAndItemSchema_BuildsAndEmitsStreamingSupport|FullyQualifiedName~Generate_WithServerVariableDefaults_UsesResolvedBaseUrl"

@HavenDV HavenDV marked this pull request as ready for review April 11, 2026 01:08
@HavenDV HavenDV merged commit dcd047f into main Apr 11, 2026
3 checks passed
@HavenDV HavenDV deleted the codex/ci-shared-generator-fixes branch April 11, 2026 01:09
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.

1 participant