Skip to content

Conversation

markstory
Copy link
Owner

No description provided.

@codecov
Copy link

codecov bot commented Jul 22, 2022

Codecov Report

Merging #366 (6a1681f) into 5.x (cf267af) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 6a1681f differs from pull request most recent head 0dbf340. Consider uploading reports for the commit 0dbf340 to get more accurate results

@@            Coverage Diff            @@
##                5.x     #366   +/-   ##
=========================================
  Coverage     92.39%   92.39%           
  Complexity      116      116           
=========================================
  Files            10       10           
  Lines           342      342           
=========================================
  Hits            316      316           
  Misses           26       26           
Impacted Files Coverage Δ
src/AssetScanner.php 90.00% <100.00%> (ø)
src/Factory.php 95.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 817438d...0dbf340. Read the comment docs.

I think PHPStan is wrong here as other tools are able to figure out what
is going on, and the assetcompress scanner *is* a subclass of the
mini-asset one.
@markstory markstory merged commit de214d1 into 5.x Jul 25, 2022
@markstory markstory deleted the mini-asset-2x branch July 25, 2022 02:54
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.

1 participant