webpack / webpack-dev-middleware

A development middleware for webpack
MIT License
2.5k stars 375 forks source link

fix: assetsInfo may be undefined #1927

Closed inottn closed 3 months ago

inottn commented 3 months ago

This PR contains a:

Motivation / Use-Case

Rspack does not yet support assetsInfo, so we may need to check if assetsInfo exists.

related pr: #1924 related issues: https://github.com/web-infra-dev/rspack/issues/7614 https://github.com/web-infra-dev/rspack/issues/7618

Breaking Changes

Additional Info

alexander-akait commented 3 months ago

@inottn Can you add a small comment about this fix for rspack?

inottn commented 3 months ago

@inottn Can you add a small comment about this fix for rspack?

done

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.16%. Comparing base (0034dd9) to head (995f8fb). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1927 +/- ## ======================================= Coverage 96.16% 96.16% ======================================= Files 13 13 Lines 860 861 +1 Branches 247 248 +1 ======================================= + Hits 827 828 +1 Misses 33 33 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.