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

Enable compatibility with old and new image sharp versions. #427

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

Conversation

TonyValenti
Copy link

(Only NET60 and above use the new ImageSharp.)
Rebase and Cleanup commits.

(Only NET60 and above use the new ImageSharp.)
Rebase and Cleanup commits.
@TonyValenti
Copy link
Author

Hi @ststeiger Would you please give this code a review? I've cleaned things up quite a bit since the original PR and rebased on your latest commit. The changes are:

  • Based on your latest commit
  • Quite minimal
  • Only use new ImageSharp versions for recent .NET versions

I really expect it would only take a few minutes of your time to review and compare.

Litify
Abacus Fix
TimeSlips fix
NEOS Fix
eBillityFix
@chrisg32 chrisg32 mentioned this pull request Apr 11, 2024
Litify, Prevail, TimeSolv, Clio.
@subahan-kothawal
Copy link

@ststeiger can you please review these changes and merge. we indeed require these update.

@TonyValenti
Copy link
Author

Hi @ststeiger ! Welcome back! I just resolved the conflicts so you can merge this.

@AndreasBurbach
Copy link

hey there,
pls update to higher version because of vulnerability:
GHSA-65x7-c272-7g7r

@peekyou
Copy link

peekyou commented Dec 12, 2024

Hello @ststeiger, any plan to merge this PR?

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.

4 participants