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

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

Open
pixiandouban opened this issue Jun 5, 2022 · 13 comments
Labels
help wanted Extra attention is needed

Comments

@pixiandouban
Copy link

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

@l0o0
Copy link
Owner

l0o0 commented Jun 13, 2022

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

@zepinglee
Copy link
Contributor

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

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

@l0o0
Copy link
Owner

l0o0 commented Oct 13, 2022

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

@zepinglee
Copy link
Contributor

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

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

@l0o0
Copy link
Owner

l0o0 commented Mar 27, 2023

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

@l0o0 l0o0 changed the title 是否逐步合并到Zotero主线 是否逐步合并到Zotero主线[关于与官方仓库合并问题汇总] Mar 27, 2023
@l0o0
Copy link
Owner

l0o0 commented Mar 27, 2023

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

@jiaojiaodubai
Copy link
Collaborator

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

@l0o0
Copy link
Owner

l0o0 commented Nov 29, 2023

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

@jiaojiaodubai
Copy link
Collaborator

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

@l0o0
Copy link
Owner

l0o0 commented Nov 29, 2023

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

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

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

@jiaojiaodubai
Copy link
Collaborator

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

@l0o0
Copy link
Owner

l0o0 commented Dec 11, 2023

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

@jiaojiaodubai
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants