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

Prepare repo for .NET 8 #6810

Merged
merged 51 commits into from
Aug 17, 2023
Merged

Prepare repo for .NET 8 #6810

merged 51 commits into from
Aug 17, 2023

Conversation

bording
Copy link
Member

@bording bording commented Aug 10, 2023

This PR makes the following changes to prepare for .NET 8:

  • The major version has been incremented to 9
    • Obsoletes have been adjusted to account for this new major version
  • Target framework has been changed to net8.0
  • Code has been adjusted due to dropping net472
    • #ifdefs based on NETFRAMEWORK have been removed
    • Concepts that don't exist in .NET have been removed. For example, serialization and AppDomains
  • Dependencies have been updated to prerelease versions for .NET 8
  • The analyzer project and its dependencies have been updated since we can assume VS 2022 as a minimum version now.
  • init.ps1 has been removed since dropping net472 means there's no more chance of packages.config files being used to install the package.

In addition, changes were made to PreObsoleteAttribute to make it easier to use and ensure that appropriate context is provided around why the future obsoletions have been staged.

@andreasohlund andreasohlund self-assigned this Aug 11, 2023
src/NServiceBus.Core/Pipeline/PipelineComponent.cs Outdated Show resolved Hide resolved
src/NServiceBus.Core/Pipeline/RegisterStep.cs Outdated Show resolved Hide resolved
src/NServiceBus.Core/Receiving/ReceiveComponent.cs Outdated Show resolved Hide resolved
src/NServiceBus.Core/Sagas/Sagas.cs Outdated Show resolved Hide resolved
src/NServiceBus.Core/Sagas/Sagas.cs Outdated Show resolved Hide resolved
src/NServiceBus.Core/Sagas/Sagas.cs Outdated Show resolved Hide resolved
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.

3 participants