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: init 命令生成js类型的导出文件,以支持纯js和jsx项目的使用 #129

Merged
merged 5 commits into from
Mar 23, 2023

Conversation

cuiyueshuai
Copy link
Contributor

@cuiyueshuai cuiyueshuai commented Mar 15, 2023

先说一下大体改动:

  1. 我删除掉了原init命令里面的默认值 { isDefault: true } , 我查看了一下全局代码,感觉并没有使用到这个默认值的操作,如果不正确,希望和我说一下,不能这样改,我考虑看看其他方式。
  2. 给命令添加了可选参数 [type], 以支持js和ts类型的文件生成,
  3. 并对默认项目配置 加了个参数 fileType 以支持 生成文件类型,
    但是感觉这部分改动还挺大的,对原设计代码上,所以您应该先看看,我的意图是,纯js和jsx项目中,生成ts类型的文件导出,会引起编译错误的,虽然我可以去手动改ts后缀,但是总归不太方面,而且我不太想为了这么几个文件就改动编译配置,让我项目工程就开始支持编译ts文件,所以希望一开始就只为了js和jsx项目服务,没有ts的相关引入和操作

@linhuiw
Copy link
Member

linhuiw commented Mar 17, 2023

我加你微信了,我们视频会议沟通下改动吧。

@cuiyueshuai
Copy link
Contributor Author

cuiyueshuai commented Mar 18, 2023

@linhuiw 那个我没收到你的微信请求呀,你加错了吧,我微信号已经通过邮箱发送给你,或者我如何联系你?

邮件标题:
image

@linhuiw
Copy link
Member

linhuiw commented Mar 21, 2023

LGTM,代码已经Review。

@cuiyueshuai 请帮忙补全 init 的 README 文档

@linhuiw linhuiw changed the title feat: init 命令生成js类型的导出文件,以支持纯js和jsx项目的使用 WIP: feat: init 命令生成js类型的导出文件,以支持纯js和jsx项目的使用 Mar 22, 2023
@cuiyueshuai
Copy link
Contributor Author

@linhuiw 文档已经完成更新啦,你在check一下,辛苦

@linhuiw linhuiw changed the title WIP: feat: init 命令生成js类型的导出文件,以支持纯js和jsx项目的使用 feat: init 命令生成js类型的导出文件,以支持纯js和jsx项目的使用 Mar 23, 2023
@linhuiw linhuiw merged commit a0ea9a7 into alibaba:master Mar 23, 2023
@linhuiw
Copy link
Member

linhuiw commented Mar 23, 2023

已合并,并发布版本 1.1.0

非常感谢 PR @cuiyueshuai

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.

None yet

2 participants