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

Rewrite v2 #543

Closed
wants to merge 5 commits into from
Closed

Rewrite v2 #543

wants to merge 5 commits into from

Conversation

Close

- #245
- #420
- #423
- #344
- #342
- #432
- #436
- #331
- #324
- #311
- #297
- #296
- #286
- #282
- #280
- #239
- #172
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@3a93f5a). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #543   +/-   ##
=========================================
  Coverage          ?   94.28%           
=========================================
  Files             ?        1           
  Lines             ?       70           
  Branches          ?       30           
=========================================
  Hits              ?       66           
  Misses            ?        4           
  Partials          ?        0           

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 3a93f5a...8cbc7dc. Read the comment docs.

@afc163
Copy link
Contributor

afc163 commented Nov 14, 2020

@sorrycc 来看看?

@sorrycc
Copy link
Member

sorrycc commented Nov 24, 2020

#459 的处理方式一样,丢功能了。换这种方式,内部是不敢直接在框架里直接升的。

@sorrycc
Copy link
Member

sorrycc commented Nov 24, 2020

改名和顺序的问题,不换方案应该也是有解的。

@yesmeck
Copy link
Contributor Author

yesmeck commented Nov 24, 2020

不行我换个名字发个新包

@yesmeck
Copy link
Contributor Author

yesmeck commented Nov 24, 2020

一个想法,umi 里可以同时使用新旧两个版本,可以先用新版本转换一遍,再用旧版本去兜底 import * as antd from 'antd'; 的情况,并显示警告,后面 umi 升大版本再把旧版 babel-plugin-import 下掉。

@yesmeck yesmeck closed this Dec 9, 2021
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.

4 participants