Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: webpack compilations fail when using filesystem cache, #458 #459

Closed

Conversation

webdiscus
Copy link

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

There is the Bugfix for the issue Webpack compilations fail when using filesystem cache.

Breaking Changes

Additional Info

CHANGELOG.md Outdated
@@ -2,6 +2,13 @@

All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.

### [4.1.2](https://github.com/webpack-contrib/image-minimizer-webpack-plugin/compare/v4.1.1...v4.1.2) (2024-12-16)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelog is auto generated so dont modify that

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, i will correct it now...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have removed changes in package.json and CHANGELOG.md

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "image-minimizer-webpack-plugin",
"version": "4.1.1",
"version": "4.1.2",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexander-akait is the one bumping and publishing versions, so dont include that in this pr

@webdiscus webdiscus force-pushed the fix-cache-filesystem2 branch from c213a30 to b6fbf39 Compare December 16, 2024 12:41
@evenstensberg
Copy link
Member

Thx. Could you write a reproduction repo with the bug, so we can compare results? I see that you've just added an extra check if resourcepath doesnt contain the data.

@webdiscus
Copy link
Author

Thx. Could you write a reproduction repo with the bug, so we can compare results? I see that you've just added an extra check if resourcepath doesnt contain the data.

in the issue #458 is the link to the repo:

cache-filesystem-rebuild-image-minimizer

The steps "How to reproduce the bug" are described in the #458

@evenstensberg
Copy link
Member

great. thank you!

Copy link

codecov bot commented Dec 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.85%. Comparing base (3a924e4) to head (b6fbf39).
Report is 22 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #459      +/-   ##
==========================================
+ Coverage   77.16%   77.85%   +0.68%     
==========================================
  Files           4        4              
  Lines         727      754      +27     
  Branches      282      303      +21     
==========================================
+ Hits          561      587      +26     
- Misses        137      138       +1     
  Partials       29       29              

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

@alexander-akait
Copy link
Member

Do it better - #460, anyway big thank you

@webdiscus
Copy link
Author

@alexander-akait

after update to v4.1.2 the bug is fixed, works fine after rebuilds.
Thank you!

@webdiscus webdiscus deleted the fix-cache-filesystem2 branch December 18, 2024 17:32
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