-
Notifications
You must be signed in to change notification settings - Fork 12
Release 6.0 #18
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
Release 6.0 #18
Conversation
Update dbup-core to 6.0 and other references
# Conflicts: # src/dbup-postgresql/dbup-postgresql.csproj
Hi @droyad . When do you plan to merge this PR and release v6? Apparently, current dbup-postgresql is using some very old transitive dependencies and with .NET 9 switching to all we now get warnings below during the build: |
# Conflicts: # src/Tests/ApprovalFiles/NoPublicApiChanges.Run.Net.verified.cs
* fix: #680 PostgreSQL statements split implemented Co-authored-by: shokurov <[email protected]> * Updated approval file --------- Co-authored-by: shokurov <[email protected]> Co-authored-by: Robert Wagner <[email protected]>
|
Hello, |
@nvierge-addactis 🤦 , the package should be published shortly |
I have found an issue. Here is a PR: #27. |
* fix(#26): avoid NullReferenceException when creating database * Updated approval file --------- Co-authored-by: Robert Wagner <[email protected]>
I have found another issue in the parser. Here is a PR: #28 |
Hi @droyad, any updates on when you intend for version 6 of the library to be released? I'm looking forward to leveraging the ability to use an ILogger to capture logs 😄 |
Hello @droyad , |
@nvierge-addactis Thanks for the heads up, the Action failed. It's published now as |
Breaking Changes
https://dbup.readthedocs.io/en/latest/breaking-changes/