Skip to content

Fix for webpack 5 + babel "failed to resolve" error #20810

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

Merged
merged 1 commit into from
May 18, 2021

Conversation

buu700
Copy link
Contributor

@buu700 buu700 commented May 15, 2021

Addresses #20800.

I don't know if this is the best fix for this issue, but seems to be an effective temporary workaround at least. It's possible that a bug may need to be fixed in babel-loader or @babel/runtime.

@google-cla

This comment has been minimized.

@google-cla google-cla bot added the cla: no label May 15, 2021
@buu700
Copy link
Contributor Author

buu700 commented May 15, 2021

@googlebot I signed it!

Copy link

@dangrussell dangrussell left a comment

Choose a reason for hiding this comment

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

This solves the issue for me!

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

Hi @buu700,

Can you please fix the validation CI by updating your commit to;

fix(@angular-devkit/build-angular): correctly resolve babel runtime helpers

Closes #20800

@alan-agius4 alan-agius4 added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 17, 2021
@buu700
Copy link
Contributor Author

buu700 commented May 17, 2021

Done, thanks @alan-agius4!

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

Thanks LGTM

@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels May 17, 2021
@alan-agius4 alan-agius4 merged commit d34dc88 into angular:master May 18, 2021
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jun 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants