Skip to content

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Oct 8, 2022

What kind of change does this PR introduce?
chore

Did you add tests for your changes?
No.

If relevant, did you update the documentation?
No.

Summary

Update interpret dependency to v3.

Does this PR introduce a breaking change?

https://github.com/gulpjs/interpret/releases/tag/v3.0.0
https://github.com/gulpjs/rechoir/releases/tag/v0.8.0

Other information
No

@snitin315 snitin315 requested a review from a team as a code owner October 8, 2022 02:51
@snitin315 snitin315 changed the title chore: udpate interpret to v3 chore: udpate interpret to v3 and rechoir to v0.8 Oct 8, 2022
@codecov
Copy link

codecov bot commented Oct 8, 2022

Codecov Report

Merging #3437 (8a06333) into next (bab5b27) will decrease coverage by 0.44%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #3437      +/-   ##
==========================================
- Coverage   91.74%   91.30%   -0.45%     
==========================================
  Files          22       22              
  Lines        1587     1587              
  Branches      447      447              
==========================================
- Hits         1456     1449       -7     
- Misses        131      138       +7     
Impacted Files Coverage Δ
packages/webpack-cli/src/webpack-cli.ts 92.69% <0.00%> (-0.70%) ⬇️

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 bab5b27...8a06333. Read the comment docs.

Comment on lines -12 to -14
expect(stderr).toContain("Cannot find module 'iced-coffee-script/register'");
expect(stderr).toContain("Cannot find module 'iced-coffee-script'");
expect(stderr).toContain("Please install one of them");
Copy link
Member Author

Choose a reason for hiding this comment

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

They removed support for legacy modules.

@alexander-akait alexander-akait merged commit 0f7da89 into next Oct 8, 2022
@alexander-akait alexander-akait deleted the update-interpret branch October 8, 2022 12:41
@alexander-akait
Copy link
Member

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants