-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 all projects to .NET6.0 #2616
Conversation
@sfmskywalker CI .NET SDK does not support targeting .NET 6.0 |
@yopichy Thank you for this! I'll update the CI settings. |
@yopichy Question about this PR: will .NET 5 project still be able to use the Elsa core libraries, Entity Framework etc? |
@sfmskywalker |
@yopichy We should definitely keep support for .NET 5 for the time being. We should be able to conditionally target both versions of e.g. EFCore.BulkExtensions, right? E.g. <ItemGroup Condition=" '$(TargetFramework)' == 'net6.0'">
<PackageReference Include="EFCore.BulkExtensions" Version="6.x" />
</ItemGroup>
<ItemGroup Condition=" '$(TargetFramework)' == 'net5.0'">
<PackageReference Include="EFCore.BulkExtensions" Version="5.x" />
</ItemGroup>
<ItemGroup Condition=" '$(TargetFramework)' == 'netcoreapp3.1'">
<PackageReference Include="EFCore.BulkExtensions" Version="3.x" />
</ItemGroup> |
@sfmskywalker ok i'll try it |
@yopichy Thank you 🙏🏻 |
Done |
src/activities/Elsa.Activities.Http/Elsa.Activities.Http.csproj
Outdated
Show resolved
Hide resolved
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.
Some changes are required to maintain 3.1 support
src/activities/Elsa.Activities.Temporal.Common/Elsa.Activities.Temporal.Common.csproj
Outdated
Show resolved
Hide resolved
...hooks.Persistence.EntityFramework.Core/Elsa.Webhooks.Persistence.EntityFramework.Core.csproj
Show resolved
Hide resolved
...istence.EntityFramework.SqlServer/Elsa.Webhooks.Persistence.EntityFramework.SqlServer.csproj
Show resolved
Hide resolved
...s.Persistence.EntityFramework.Sqlite/Elsa.Webhooks.Persistence.EntityFramework.Sqlite.csproj
Show resolved
Hide resolved
...amples/aspnet/Elsa.Samples.ContextualWorkflowHttp/Elsa.Samples.ContextualWorkflowHttp.csproj
Outdated
Show resolved
Hide resolved
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.
minimal changes to get monolith to start running on 3.1, additional changes still required
...ework/Elsa.Persistence.EntityFramework.Sqlite/Elsa.Persistence.EntityFramework.Sqlite.csproj
Show resolved
Hide resolved
@mohdali @sfmskywalker i'll updated according to suggestion ASAP |
Remove common Authorization package
reimplement fix for #1568
@yopichy thank you very much. I went ahead and pushed some updates to implement some of the changes. hope it helps. |
Thanks for the help @mohdali |
This major change affect the Elsa 2.x only? |
More specifically, it only affects |
No description provided.