alibaba / kiwi

🐤 Kiwi-国际化翻译全流程解决方案
2.5k stars 229 forks source link

feat: init 和extract 完全支持纯js或jsx项目的使用与替换 #128

Closed cuiyueshuai closed 1 year ago

cuiyueshuai commented 1 year ago
CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

cuiyueshuai commented 1 year ago

init的相关命令可能改动比较大,我觉得这个影响挺大的,虽然想这样做,但是对原项目跨度区别会不会太大,如果有,希望可以重新指点一下,因为常规纯js项目肯定是没有引入ts相关编译依赖的,如果生成ts的导出文件,等于手动要去修改相关文件后缀的,而且手动改,就不符合全自动的意图了,项目咋没有一些逻辑单元测试之类的案例,我看见没有,所以就没做测试了

cuiyueshuai commented 1 year ago

希望大佬早点看看,讨论一下怎么改,我在我自己的具体项目中完成了相关中文提取操作,然后使用了一遍,但是现在因为是源代码,所以写项目里的命令还要link我的改动源代码,太不方便,发布具体npm包就不会了,我的其他同事都可以用

Carrotzpc commented 1 year ago

这 pr 提的不太讲究,可以包含下 #119 中的 commit,自己直接抄过来都不在 pr 里提一句

cuiyueshuai commented 1 year ago

@Carrotzpc 大兄弟说的对,我只是在人家那个PR里发言了一下,其他人没看到确实不好,说明一下,这部分PR改动包含另一位兄弟的代码,我觉得他写的很好,直接就用了,看 https://github.com/alibaba/kiwi/pull/119即可了解经过,可惜了,不知道为啥没人维护,只能本地安装npm包使用了

linhuiw commented 1 year ago

@cuiyueshuai #119 的PR 已经合并了,你可以基于最新的 master 改动一下这个 PR 的代码,修复一下代码冲突。

辛苦了

cuiyueshuai commented 1 year ago

相关合并我解决了冲突提了新的PR,请查看新PR吧,https://github.com/alibaba/kiwi/pull/129