-
Notifications
You must be signed in to change notification settings - Fork 170
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
refactor: lodash-es instead of lodash #1740
base: develop
Are you sure you want to change the base?
Conversation
@@ -2,7 +2,7 @@ | |||
import fs from 'fs'; | |||
import path from 'path'; | |||
import matter from 'gray-matter'; | |||
import camelCase from 'lodash/camelCase'; | |||
import camelCase from 'camelcase'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里为啥不直接 import { camelCase } from 'lodash-es' 了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我换过,换了会报错,site 打包不支持 es
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
明白了
🤔 这个 PR 的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
Tencent/tdesign-vue-next#4961
common 仓库 lodash 已改 lodash-es
Tencent/tdesign-common#2040
📝 更新日志
fix(组件名称): 处理问题或特性描述 ...
本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单