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

2.8.0 version has a error by webpack babel loader #5230

Closed
grandEarshot opened this issue Mar 9, 2017 · 8 comments
Closed

2.8.0 version has a error by webpack babel loader #5230

grandEarshot opened this issue Mar 9, 2017 · 8 comments
Assignees
Labels
🐛 Bug Ant Design Team had proved that this is a bug.

Comments

@grandEarshot
Copy link

Environment(required)

  • antd version:2.8.0
  • OS and its version:win10
  • Browser and its version:chrome

What did you do? Please provide steps to re-produce your problem.

when I use webpack has a error

What do you expected?

Dont has this problem when i use 2.7.x version

What happen?

ERROR in . /~/ antd /lib/carousel/index.js
Module not found: Error: Cannot resolve module 'lodash.debounce' in xxx\node_modules\antd \lib\carousel

@ ./~/antd/lib/carousel/index.js 28:14-40

Re-producible online demo

@afc163 afc163 self-assigned this Mar 9, 2017
@afc163 afc163 added the 🐛 Bug Ant Design Team had proved that this is a bug. label Mar 9, 2017
@afc163 afc163 closed this as completed in 73abb1a Mar 9, 2017
@afc163
Copy link
Member

afc163 commented Mar 9, 2017

Sorry for missing that 😓

@afc163
Copy link
Member

afc163 commented Mar 9, 2017

@yesmeck Do we have any way to prevent dependencies missing?

@yesmeck
Copy link
Member

yesmeck commented Mar 9, 2017

We can only install production dependencies and antd-tools on CI when we run npm run dist.

@yesmeck
Copy link
Member

yesmeck commented Mar 9, 2017

Weird, antd#react-lazy-load also depends on lodash.debounce, it should be installed.

@yesmeck
Copy link
Member

yesmeck commented Mar 9, 2017

@afc163 Can no find a good way to prevent this...

@yesmeck
Copy link
Member

yesmeck commented Mar 9, 2017

More carefully when code review..

@afc163
Copy link
Member

afc163 commented Mar 10, 2017

eslint can do it but it does not support TypeScript yet.

@lock
Copy link

lock bot commented May 1, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators May 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐛 Bug Ant Design Team had proved that this is a bug.
Projects
None yet
Development

No branches or pull requests

3 participants