Skip to content
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

Update to EFCore 9 #292

Merged
merged 18 commits into from
Nov 18, 2024
Merged

Update to EFCore 9 #292

merged 18 commits into from
Nov 18, 2024

Conversation

Smalls1652
Copy link
Owner

@Smalls1652 Smalls1652 commented Nov 13, 2024

Description

  • Updates EFCore to version 9.

Related issues

  • None

Stack

@Smalls1652 Smalls1652 added enhancement New feature or update to an existing feature dependencies Pull requests that update a dependency file Libraries Related to core libraries labels Nov 13, 2024
@Smalls1652 Smalls1652 self-assigned this Nov 13, 2024
@github-actions github-actions bot mentioned this pull request Nov 13, 2024
@Smalls1652
Copy link
Owner Author

This will take some extra work to get updated due to a lot of breaking changes with the CosmosDB provider.

I can either take this opportunity to make the jump to Postgres like I've been planning for a while (Since I'm also planning on moving MuzakBot exclusively into my private server infrastructure) or just fix the breaking changes for now.

@Smalls1652 Smalls1652 changed the base branch from update/lgtm-dotnet-9 to update/container-images-dotnet-9 November 13, 2024 05:08
Base automatically changed from update/container-images-dotnet-9 to main November 13, 2024 05:22
@Smalls1652
Copy link
Owner Author

Smalls1652 commented Nov 14, 2024

This will be merged after the next major update to accommodate for the migration from CosmosDB to Postgres. Primarily due to the reliance on the EFCore 8 CosmosDB provider being needed to easily migrate the data over.

@github-actions github-actions bot mentioned this pull request Nov 15, 2024
* Remove multiple items

- Removed AlbumReleaseDbContext.
- Removed LyricsAnalyzerDbContext.
- Removed CosmosDB to Postgres migrations.
- Removed CosmosDB as a database type.

* Remove CosmosDB to Postgres migration in main bot app

* Remove CosmosDB packages
@Smalls1652
Copy link
Owner Author

Just about everything is ready to go, except for Npgsql's EFCore 9 update. Once that's published, I can update the package reference in this branch and I should be able to merge this.

@Smalls1652 Smalls1652 merged commit 6f368aa into main Nov 18, 2024
4 checks passed
@Smalls1652 Smalls1652 deleted the update/efcore-9 branch November 18, 2024 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or update to an existing feature Libraries Related to core libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant