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

Drop support for .NET Framework 4.5.1 #351

Merged
merged 1 commit into from
Mar 9, 2023
Merged

Conversation

0xced
Copy link
Member

@0xced 0xced commented Feb 17, 2023

Support for .NET Framework 4, 4.5, and 4.5.1 ended on January 12, 2016, that was 7 years ago!

Also bump Microsoft.Extensions.Configuration.Binder and Microsoft.Extensions.DependencyModel to version 6.0.0 since version 2.0.0 and 3.0.0 respectively have been deprecated as part of the .NET Package Deprecation effort.

Finally, this change allows to remove all conditional compilation.

Note that removing a target framework is a breaking change so it will require a major version bump, i.e. 4.0.0.

Support for .NET Framework 4, 4.5, and 4.5.1 [ended on January 12, 2016](https://learn.microsoft.com/en-GB/lifecycle/faq/dotnet-framework), that was 7 years ago!

Also bump Microsoft.Extensions.Configuration.Binder and Microsoft.Extensions.DependencyModel to version 6.0.0 since version 2.0.0 and 3.0.0 respectively have been deprecated as part of the [.NET Package Deprecation effort](dotnet/announcements#217).

Finally, this change allows to remove all conditional compilation.

Note that removing a target framework is a breaking change so it will require a major version bump, i.e. 4.0.0.
@HexHunter97
Copy link

Serilog itself has also dropped support for anything below .Net Framework 4.6.1 and .Net Standard 2.0 some time ago, so there is no expectation from that angle to keep supporting 4.5.1 either.

@glen-84
Copy link

glen-84 commented Feb 18, 2023

4.6.1 is also EOL, why support it?

Edit: I suppose bc the main project still supports it.

@andrewboudreau
Copy link

Will this go into a 3.5 pre-release or is it too late for that?

@nblumhardt nblumhardt merged commit 1b5feb2 into serilog:dev Mar 9, 2023
@nblumhardt
Copy link
Member

We should look at releasing this as 4.0.0 in the very near future 👍

@andrewboudreau
Copy link

I actually built this branch out and getting the same warning with 6.0.0 that I was with 3.0.0. I had to go to 7. I assume there is some nugget switch for >= on DependencyModel and Configuration Binder

@glen-84
Copy link

glen-84 commented Mar 9, 2023

@0xced Any reason not to update Microsoft.Extensions.* to 7.0.0?

@glen-84
Copy link

glen-84 commented Mar 9, 2023

According to this, the EOL of these packages might be bound to the corresponding framework, so perhaps 6.0.0 is best.

Having said that, we still need a solution for #331 (comment).

Does that lead us back to #339?

@andrewboudreau
Copy link

@glen-84 end of the day the issues ends up being build images available on appveyor for net7?

@andrewboudreau
Copy link

Additionally, I'm running the dependency with 3.0 on other 7.0 by ignoring the warning. Things seem to to be fine. So, 6.0 probably fine?

@0xced 0xced deleted the drop-net451 branch March 9, 2023 19:05
@glen-84
Copy link

glen-84 commented Mar 12, 2023

Ignoring the warning is not a solution.

@glen-84
Copy link

glen-84 commented Mar 27, 2023

We should look at releasing this as 4.0.0 in the very near future 👍

@nblumhardt How near is very near? 🙂

@nblumhardt nblumhardt mentioned this pull request May 10, 2023
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.

6 participants