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

⬆️ Upgrade to ImageSharp 3 #75

Merged
merged 1 commit into from
Mar 12, 2024
Merged

⬆️ Upgrade to ImageSharp 3 #75

merged 1 commit into from
Mar 12, 2024

Conversation

tomrijnbeek
Copy link
Member

✨ What's this?

Upgrades ImageSharp to 3.1.3. This includes updating some method calls to their new signatures.

🔗 Relationships

See #74

🔍 Why do we want this?

There is a security vulnerability in ImageSharp 2 and it keeps trying to tell me everywhere 😛

🏗 How is it done?

Updated the package, then resolved all build errors.

💥 Breaking changes

Might break compatibility with other people who use ImageSharp 2 still. TD uses 3 though.

🔬 Why not another way?

Need to bump it somehow.

💡 Review hints

I've run a clean build of TD with this Bearded.Graphics version and it builds and runs fine. Things are drawn as expected.

@tomrijnbeek tomrijnbeek added dependencies Pull requests that update a dependency file breaking-change labels Mar 12, 2024
@tomrijnbeek tomrijnbeek requested a review from paulcscharf March 12, 2024 18:42
@paulcscharf paulcscharf merged commit 881a614 into master Mar 12, 2024
1 check passed
@paulcscharf paulcscharf deleted the imagesharp-3 branch March 12, 2024 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants