l0o0 / translators_CN

Zotero translator中文网页抓取翻译器🎉This is Zotero translators for Chinese Sites(beta), not the official Zotero repo
GNU Affero General Public License v3.0
4.11k stars 525 forks source link

是否逐步合并到Zotero主线[关于与官方仓库合并问题汇总] #113

Open pixiandouban opened 2 years ago

pixiandouban commented 2 years ago

合并到主线上可以让其他在使用Zotero的在线平台使用,现在只能线下使用。

l0o0 commented 2 years ago

是的同学,合并到主线,会减少很多不必要的翻译器更新步骤。不过我之前提交了几个pr好像没有动静。可能官方代码风格要求比较严格😀

zepinglee commented 2 years ago

可能官方代码风格要求比较严格😀

是因为 npm run lint -- "Your Translator.js" 报告的问题吗?

l0o0 commented 2 years ago

是ESlint 代码风格的问题。我最近想重新按照官方的代码风格进行更新。如果可以的话,我们可以专门开一个帖子专门说明一下,转换器相关的配置和代码风格事项

zepinglee commented 2 years ago

是ESlint 代码风格的问题。我最近想重新按照官方的代码风格进行更新。如果可以的话,我们可以专门开一个帖子专门说明一下,转换器相关的配置和代码风格事项

OK,我可以参与。虽然我也接触 JS 不多,但是前不久刚把 PKULaw.js 改成符合 ESlint 要求的格式。

l0o0 commented 1 year ago

最近转换器代码有变化,提供了await 这种异步函数,提供了许多接口用来取代 doGetdoPost,还有其他方法上的更新。可用的函数可以参考这个文件中的规范 https://github.com/zotero/translators/blob/master/index.d.ts ,官方现在应该对新转换器,要求用新版函数 20230325 又提交了一个request https://github.com/zotero/translators/pull/3002

l0o0 commented 1 year ago

来自官方仓库的殷切呼吁,我们也在尝试往官方仓库提出合并请求。

45

jiaojiaodubai commented 11 months ago

@pixiandouban @zepinglee @l0o0 ,我在我的LintIt分支整理了需要进一步完善的翻译器,带有Lint All translators提交记录的翻译器是可以通过npm run lint -- "xxx.js" --fix自动修复的,其他需要手动修复的记录在README.md

l0o0 commented 11 months ago

@jiaojiaodubai 非常优秀。中文库中已有的转换,也可以按照新的转换器标准更新一下,后面可以推送到官方仓库

jiaojiaodubai commented 11 months ago

@l0o0 有些翻译器可能需要原作者的帮助才能继续改进,因为我没有测试环境(平台帐号)或者抓取本身需要做改动(有的翻译器使用了eval(),这在Lint规则中是禁止的)

l0o0 commented 11 months ago

@l0o0 有些翻译器可能需要原作者的帮助才能继续改进,因为我没有测试环境(平台帐号)或者抓取本身需要做改动(有的翻译器使用了eval(),这在Lint规则中是禁止的)

我觉得这些应该不是问题。刚开始我本着能用就行的原则,并不会对提交的翻译器提很多要求,所以代码和实现百花齐放。

对于账户我倒不用担心,我们用户基数大,应该可找出志愿者协助我们。

jiaojiaodubai commented 11 months ago

@l0o0 LintIt分支中的翻译器已经基本完成更新,剩下的需要原作者协助了(详见LintItREADME.MD)。 知网、bilibili、读秀、文津进行了比较大的改动,从最近的反馈来看,新版知网翻译器已经比较稳定了。

l0o0 commented 11 months ago

@jiaojiaodubai 感谢你的更新工作,知网的转换器可以开PR合并了

jiaojiaodubai commented 10 months ago

大部分翻译器已经更新完毕,部分未更新的翻译器可能需要其他朋友来弄了,具体清单详见LintIt分支的README.md