Skip to content

Conversation

alexander-akait
Copy link
Member

@alexander-akait alexander-akait commented Jan 17, 2022

This PR contains a:

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

Motivation / Use-Case

fixes #900

Breaking Changes

No

Additional Info

Also fix bug with relative paths for hot code, it was broken before...

@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #901 (26b7d9d) into master (e4f4c1e) will decrease coverage by 0.93%.
The diff coverage is 82.60%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #901      +/-   ##
==========================================
- Coverage   91.23%   90.29%   -0.94%     
==========================================
  Files           5        5              
  Lines         821      824       +3     
  Branches      219      220       +1     
==========================================
- Hits          749      744       -5     
- Misses         66       70       +4     
- Partials        6       10       +4     
Impacted Files Coverage Δ
src/index.js 96.27% <82.22%> (+0.02%) ⬆️
src/loader.js 88.95% <100.00%> (ø)
src/utils.js 57.31% <0.00%> (-9.76%) ⬇️

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 e4f4c1e...26b7d9d. Read the comment docs.

@alexander-akait alexander-akait merged commit dfb9afd into master Jan 17, 2022
@alexander-akait alexander-akait deleted the fix-types branch January 17, 2022 20:01
@alexander-akait
Copy link
Member Author

@barak007 I will merge it and do release, feel free to improve any types (if you found a typo/wrong types/somethings else)

};
type TODO = any;
type CssModule = import("webpack").Module & {
content: Buffer;

Choose a reason for hiding this comment

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

@alexander-akait can this content accept string or Buffer is the only possible value?

Copy link
Member Author

Choose a reason for hiding this comment

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

We have content: Buffer.from(content) in our code, so I used Buffer, but webpack can accept strings too, need to tests, if it will work feel free to improve it

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