-
-
Notifications
You must be signed in to change notification settings - Fork 469
V1: Update dbup-core in .net6 and newer #1078
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Awesome! Can you update the release notes with a fitting description? |
49f9173 to
9d1b195
Compare
|
Odd that I need to continuously need to approve runs. I'll try to get that fixed |
|
@MMonrad there's some runtime errors |
I will look into as soon I have some free time :) |
df1cb60 to
6336ce6
Compare
|
@rasmus there is no stable version for postgres, so I converted this PR to draft. Will publish again when, a stable version is released. |
|
Thanks |
|
@rasmus dbup postgres has been updated, can you start the build pipeline? |
| <PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="9.0.1" /> | ||
| <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="9.0.1" /> | ||
| <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="9.0.1" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These versions originate from .NET 9
| <PackageReference Include="Npgsql" Version="8.0.6" /> | ||
| <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="8.0.11" /> | ||
| <PackageReference Include="Npgsql" Version="9.0.2" /> | ||
| <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="9.0.1" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This version originate from .NET 9
|
Closed due to postgre package in db up would require entity framework version 9.0.1 packages or later. @rasmus I'm closing this, there is not a good package fit to support this yet. Maybe in .Net 10 I can create a new PR :) |
|
Awesome. Thanks, supporting non-LTS versions would take more time and thus a bigger community.On 9 Mar 2025, at 17.11, MMonrad ***@***.***> wrote:
Closed due to postgre package in db up would require entity framework version 9.0.1 packages or later.
@rasmus I'm closing this, there is not a good package fit to support this yet. Maybe in .Net 10 I can create a new PR :)—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
MMonrad left a comment (eventflow/EventFlow#1078)
Closed due to postgre package in db up would require entity framework version 9.0.1 packages or later.
@rasmus I'm closing this, there is not a good package fit to support this yet. Maybe in .Net 10 I can create a new PR :)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
In dbup-core version 6 and later a breaking change was made in the logger. Version 6 also includes the logger abstractions package, that does not support the old .net core or standard versions.
This PR updates dbup-core to version 6 when running .net6 or newer