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

feat: support alias config #358

Closed
wants to merge 2 commits into from
Closed

feat: support alias config #358

wants to merge 2 commits into from

Conversation

billfeller
Copy link
Contributor

基于 @rollup/plugin-alias 支持路径别名配置项

close #356

@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #358 (ef0ceba) into master (1140590) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #358      +/-   ##
==========================================
+ Coverage   74.01%   74.06%   +0.05%     
==========================================
  Files          16       16              
  Lines         481      482       +1     
  Branches      171      172       +1     
==========================================
+ Hits          356      357       +1     
  Misses        124      124              
  Partials        1        1              
Impacted Files Coverage Δ
packages/father-build/src/schema.ts 100.00% <ø> (ø)
packages/father-build/src/getRollupConfig.ts 97.75% <100.00%> (+0.02%) ⬆️

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 1140590...ef0ceba. Read the comment docs.

@billfeller
Copy link
Contributor Author

@sorrycc 帮确认下这个问题,使用 dumi 开发组件库,构建产物存在重复导出问题,需要支持路径别名配置项,看下是否支持合入

基于 @rollup/plugin-alias 支持路径别名配置项
@xyx650
Copy link

xyx650 commented Jun 10, 2021

ci 没过啊,大佬 @wadezhan 看下什么问题,很需要这个 feature 谢谢

@billfeller
Copy link
Contributor Author

ci 没过啊,大佬 @wadezhan 看下什么问题,很需要这个 feature 谢谢

之前测试同样的代码变更,有构建成功的记录,怀疑是 circleci 的问题,不是代码问题。

@billfeller
Copy link
Contributor Author

billfeller commented Jun 10, 2021

@xyx650 代码重新提交了,ci 已通过。

@xyx650
Copy link

xyx650 commented Jun 15, 2021

@wadezhan @一下管理的大佬合并一下呗😂

@zoffyzhang
Copy link

@PeachScript 冒昧艾特大佬,请问这个可以发布吗?I need this feature pretty badly.

@jgchenu
Copy link

jgchenu commented Oct 10, 2021

冲突了,没合进去,麻烦解决下

@PeachScript
Copy link
Member

PeachScript commented Aug 31, 2022

感谢贡献,father 4 已发布、支持 alias,旧版本不再迭代,PR closed

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

Successfully merging this pull request may close these issues.

father-build 构建产物重复导出问题
5 participants