Documentation page on collations and case-sensitivity #2328
Conversation
roji
force-pushed
the
Collations
branch
6 times, most recently
from
0b37ffc
to
18903e0
Compare
roji
force-pushed
the
Collations
branch
2 times, most recently
from
4f0ae57
to
7f5d0f5
Compare
entity-framework/core/miscellaneous/collations-and-case-sensitivity.md
Outdated
Show resolved
Hide resolved
entity-framework/core/miscellaneous/collations-and-case-sensitivity.md
Outdated
Show resolved
Hide resolved
| </PropertyGroup> | ||
|
|
||
| <ItemGroup> | ||
| <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="5.0.0-preview.3.20181.2" /> |
There was a problem hiding this comment.
You're too eager with your docs! ;-) These things aren't in preview 3...
There was a problem hiding this comment.
Yep, that's why this PR fails the build :) Just wanted to get this done with while everything is still fresh in mind.
We can hold off merging until preview4 is out...
entity-framework/core/miscellaneous/collations-and-case-sensitivity.md
Outdated
Show resolved
Hide resolved
|
|
||
| * [SQL Server documentation on collations](https://docs.microsoft.com/sql/relational-databases/collations/collation-and-unicode-support). | ||
| * [Microsoft.Data.Sqlite documentation on collations](https://docs.microsoft.com/dotnet/standard/data/sqlite/collation). | ||
| * [PostgreSQL documentation on collations](https://www.postgresql.org/docs/current/collation.html). |
There was a problem hiding this comment.
This will be replaced with an EFCore.PG provider doc page - the provider also supports index collations, and a database-level collation that propagates in EF Core, like identity (since the PostgreSQL support in CREATE DATABASE is unfortunately pretty bad).
entity-framework/core/miscellaneous/collations-and-case-sensitivity.md
Outdated
Show resolved
Hide resolved
|
@ajcvickers the EF.Functions.Collate support isn't in preview4 - I guess we should wait until preview5? Or do you think we should push a stripped-down version with it now? |
|
@roji Either way is fine with me. |
|
Then I say let's wait... Better to have the whole feature out. If someone asks, we can always point them to this PR... |
Change depend on 5.0.0-preview5 where previews are needed. Closes #2273
The APIs in the samples will only appear in
preview4preview5, so we can't actually merge this until then.Closes #2273