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

upgrade: .NET 7.0 #1936

Merged
merged 83 commits into from
Oct 10, 2023
Merged

upgrade: .NET 7.0 #1936

merged 83 commits into from
Oct 10, 2023

Conversation

lodicolo
Copy link
Member

@lodicolo lodicolo commented Oct 8, 2023

No description provided.

lodicolo and others added 30 commits October 7, 2023 23:06
update tests to .net 7

move Microsoft.Diagnostics.Runtime reference to Core

fix: compilation errors

change action to use dotnet instead of msbuild

specific version for actions/setup-dotnet

disable CS1591 warnings to make build easier to read

fix: more compiler errors

fix workflow

fix cache key

make bundles work for .net 7

remove fody files

fix project files(?)

workflow debugging

change workflow blob paths

use dotnet publish

also build linux
- deleted doc controller
- this remains unfixed: dotnet/efcore#24339

I was using the following and it kept overwriting the GameContextModelSnapshot in the Sqlite directory + namespace:

 dotnet ef migrations add Net7Upgrade --context MySqlGameContext --namespace Intersect.Server.Migrations.MySql.Game --output-dir Migrations/MySql/Game/ -- --databaseType MySql

somehow this was missing in the previous commit
corrected netcore migrations for mysql
…igration)

reduce number of methods doing the same thing
(so I never have to go hunting for this information again)
@lodicolo lodicolo force-pushed the main branch 3 times, most recently from 841099a to 1b8f8a8 Compare October 9, 2023 23:51
- made pull_request have a manual dispatch
- delete request_help because it's now redundant
  - removed it because I couldn't get the thing to stop failing at the end
- made arifacts upload on failure
… run it in prod they wouldn't know about this feature)
@lodicolo lodicolo merged commit 25bec0c into AscensionGameDev:main Oct 10, 2023
1 check passed
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.

2 participants