Skip to content

Conversation

@roji
Copy link
Member

@roji roji commented Sep 22, 2025

No description provided.

@roji roji requested a review from Copilot September 22, 2025 23:10
@roji roji enabled auto-merge (rebase) September 22, 2025 23:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR synchronizes the codebase to Entity Framework Core 10.0.0-rc.2.25468.104 with a partial update and a revamp of JSON primitive collections.

  • Updates type mapping tests to support new JSON primitive collection mappings
  • Adds comprehensive test coverage for temporal types, networking types, and primitive collections
  • Restructures JSON collection handling to use more efficient PostgreSQL-specific SQL generation

Reviewed Changes

Copilot reviewed 68 out of 68 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/EFCore.PG.Tests/Storage/NpgsqlTypeMappingSourceTest.cs Adds tests for primitive collection and JSON primitive collection type mappings
test/EFCore.PG.FunctionalTests/Types/Temporal/*.cs New test files for temporal types (TimeSpan, TimeOnly, DateTime variants, DateOnly)
test/EFCore.PG.FunctionalTests/Types/Numeric/*.cs New test files for numeric types (short, long, int, float, double, decimal)
test/EFCore.PG.FunctionalTests/Types/Miscellaneous/*.cs New test files for miscellaneous types (string, Guid, byte array, bool)
test/EFCore.PG.FunctionalTests/Types/Networking/*.cs New test files for networking types (macaddr, inet)
test/EFCore.PG.FunctionalTests/Types/NpgsqlTypeFixture.cs Base test fixture for type tests
test/EFCore.PG.FunctionalTests/Query/*.cs Updated SQL assertions reflecting changes in JSON primitive collection handling

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@roji roji merged commit 4405acb into npgsql:main Sep 22, 2025
16 checks passed
@roji roji deleted the Sync branch September 22, 2025 23:34
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