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

fix: fix #4 ts support #5

Merged
merged 6 commits into from
Mar 1, 2022
Merged

Conversation

gadfly3173
Copy link
Contributor

@gadfly3173 gadfly3173 commented Mar 1, 2022

@viewweiwu
Copy link
Owner

感谢 PR,是否未提交 vue3-tabs-chrome.vue.d.ts

@gadfly3173
Copy link
Contributor Author

不需要提交那个 lib打包的时候会自己生成。不过看起来这个pr还有点问题,文件名不太对

@viewweiwu
Copy link
Owner

好的,等你改完我试试看

@gadfly3173 gadfly3173 changed the title fix: #4 ts support fix: fix #4 ts support Mar 1, 2022
@gadfly3173
Copy link
Contributor Author

gadfly3173 commented Mar 1, 2022

看起来这样就可以了,不过这冲突从哪冒出来的。。。我fork的时候好像还没那条四月份的commit。我是以volar为参照的 没看过webstorm这样对不对,按说volar能认得话就是标准的

@viewweiwu viewweiwu merged commit 6655efa into viewweiwu:master Mar 1, 2022
@viewweiwu
Copy link
Owner

viewweiwu commented Mar 1, 2022

非常感谢 PR,但是我本地测试感觉编译出来的东西非常的混乱。

还是建议把我那段代码复制到本地使用。

代码暂时先回退了。

@gadfly3173
Copy link
Contributor Author

还好吧 只是多了两个.d.ts 标准的组件类型定义,element之类的生成的也是这样,我也看不懂。但是volar能看懂,可以提供模板语法提示

@viewweiwu
Copy link
Owner

会在其它目录产生 .d.ts 文件

@gadfly3173
Copy link
Contributor Author

会在其它目录产生 .d.ts 文件

因为vue-cli处理得有问题,要好点打包的话得自己写更详细的webpack脚本或者搞点vue-tsc啥的来做。所以我是用gitignore排除了。脚本用scp而不是mv也是因为没有powershell和bash通用的移动命令,可以考虑自己写个node脚本来做重命名和移动的事,不过我懒得写了

@viewweiwu
Copy link
Owner

有点意思,我回头又试了一下,感觉你这个可以,可以把这部分代码加上。

如果有更好的优化,欢迎继续提供 PR。

@viewweiwu
Copy link
Owner

viewweiwu commented Mar 3, 2022

目前暂时发布 0.3.0-beta.0,如果没有问题的话,马上发布 3.0。

yarn add vue3-tabs-chrome@0.3.0-beta.0

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.

ts support
2 participants