Skip to content

Update dependencies and remove dotnet 7 support#65

Closed
follesoe wants to merge 5 commits intomicrospaze:mainfrom
follesoe:main
Closed

Update dependencies and remove dotnet 7 support#65
follesoe wants to merge 5 commits intomicrospaze:mainfrom
follesoe:main

Conversation

@follesoe
Copy link
Copy Markdown

@follesoe follesoe commented Feb 8, 2025

This PR fixes the failing tests in ImageServiceBaseTests. It also updates external dependencies to the latest versions and removes support for dotnet 7.

Dotnet 7 was out of support in May 2024, so it can safely be removed as a target.

The goal of the PR is to keep FFImageLoading.Maui up-to-date, and prepare it for an update of the Svg.Skia dependency.

The library was pinned at an old version 1.0.0.10 of Svg.Skia, which again prevented updates to SkiaSharp in projects using this library. I have opened pull requests against the Svg.Skia updating it to SkiaSharp v3. Once/if hat PR gets merged, I will update FFImageLoading.Maui to support the new Svg.Skia package, which would also allow us to update to SkiaSharp v3 which is the end-goal.

Remove net7.0 since it is obsolete and went out of support on May 14th, 2024.
The ImageServiceBaseTests was failing due to MemoryCache either not being set on the PlatformImageLoaderTask, or by the ImageService returning a new instance of the mock cache for every access to the property.
@follesoe
Copy link
Copy Markdown
Author

follesoe commented May 6, 2025

@microspaze Some feedback/merge would be greatly appreciated 😫

@follesoe
Copy link
Copy Markdown
Author

follesoe commented May 6, 2025

If you have abandoned it I wouldn't mind being assigned contributor to the library. It's just so annoying with tons of forks and different similar NuGet package names etc. So better if we can use this repo and the existing NuGet package going forward.

@petroskin
Copy link
Copy Markdown

If you have abandoned it I wouldn't mind being assigned contributor to the library. It's just so annoying with tons of forks and different similar NuGet package names etc. So better if we can use this repo and the existing NuGet package going forward.

That would be appropriate, this library could use contributors as it's the most popular FFImageLoading successor for .NET MAUI with over 500k downloads on NuGet.

Still, the best option might be to migrate away from FFImageLoading, as was recommended more than 2 years ago.

@albyrock87
Copy link
Copy Markdown

@microspaze can this be taken care of?

@albyrock87
Copy link
Copy Markdown

Never mind, I see that Skia has been upgraded here: b1bc8a8

So we can probably close this PR.

@follesoe follesoe closed this Nov 4, 2025
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