Skip to content

Add astc files so they are checked in with LFS#56

Merged
brianpopow merged 1 commit intoSixLabors:mainfrom
Erik-White:add-astc-to-lfs
Mar 6, 2026
Merged

Add astc files so they are checked in with LFS#56
brianpopow merged 1 commit intoSixLabors:mainfrom
Erik-White:add-astc-to-lfs

Conversation

@Erik-White
Copy link
Contributor

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

Adding ASTC compression support in ImageSharp.Textures requires checking in some .astc test data files. This change just ensures that astc files are checked in with LFS and prevents changes to the attributes file in ImageSharp.Textures from being overwritten

@brianpopow brianpopow merged commit 4a5a9fe into SixLabors:main Mar 6, 2026
4 checks passed
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 19%. Comparing base (8a9aa0c) to head (7596697).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@        Coverage Diff         @@
##           main   #56   +/-   ##
==================================
  Coverage    19%   19%           
==================================
  Files         4     4           
  Lines       346   346           
  Branches     85    85           
==================================
  Hits         68    68           
  Misses      278   278           
Flag Coverage Δ
unittests 19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

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.

2 participants