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

Add net462 targeting #872

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

lahma
Copy link
Contributor

@lahma lahma commented Jul 18, 2024

It's not that much of a maintenance burden and will remove some compatibility issues for folks still on full framework. SharpZipLib 2.0 could drop support if deemed so.

Also had to update testing dependencies to get NET 8 support.

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

fixes #853
fixes #863

@lahma lahma force-pushed the net-462-support branch 7 times, most recently from 92ca51b to 0a33619 Compare July 19, 2024 07:20
Copy link

codecov bot commented Jul 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.62%. Comparing base (c51ef18) to head (2255a47).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #872      +/-   ##
==========================================
- Coverage   74.78%   74.62%   -0.16%     
==========================================
  Files          72       75       +3     
  Lines        8535     8572      +37     
==========================================
+ Hits         6383     6397      +14     
- Misses       2152     2175      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lahma
Copy link
Contributor Author

lahma commented Jul 19, 2024

@piksel can you please take a look, this PR should fix GH Actions pipelines and adds the requested net462 target which should be quite safe addition.

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.

Add support for .NET Framework 4.6.1 to target frameworks. .NET Framework 4.6.2 Support?
1 participant