Skip to content

FEAT: added more extensions for snapshot(jpg, bmp, webp) - #566

Merged
rosenbjerg merged 5 commits into
rosenbjerg:mainfrom
GorobVictor:snapshot-formats
Oct 16, 2025
Merged

rosenbjerg merged 5 commits into
rosenbjerg:mainfrom
GorobVictor:snapshot-formats

Conversation

@GorobVictor

Copy link
Copy Markdown
Contributor

Added more extensions for snapshot(jpg, bmp, webp).
Added exception when lib doesn't support extension.

@GorobVictor

Copy link
Copy Markdown
Contributor Author

@rosenbjerg, Can you check this PR?

It would be great to have a couple of image extensions in the snapshot function.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.97872% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.75%. Comparing base (1c4333e) to head (bfefa95).
⚠️ Report is 21 commits behind head on main.

Files with missing lines Patch % Lines
FFMpegCore/FFMpeg/Enums/FileExtension.cs 20.00% 4 Missing ⚠️
FFMpegCore/FFMpeg/Enums/Enums.cs 84.61% 1 Missing and 1 partial ⚠️
FFMpegCore/FFMpeg/FFMpeg.cs 90.90% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #566      +/-   ##
==========================================
+ Coverage   71.20%   71.75%   +0.54%     
==========================================
  Files         117      117              
  Lines        2973     2988      +15     
  Branches      284      282       -2     
==========================================
+ Hits         2117     2144      +27     
+ Misses        751      741      -10     
+ Partials      105      103       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rosenbjerg

Copy link
Copy Markdown
Owner

Hey @GorobVictor
Got the CI fixed and updated this branch
Looks good, thanks for the contribution

@rosenbjerg
rosenbjerg merged commit 53b3e0f into rosenbjerg:main Oct 16, 2025
3 checks passed
@VictorHorobchuk

Copy link
Copy Markdown

Hello @rosenbjerg
You're welcome!

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