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(Gadget/prism): 新的代码高亮工具 #443

Merged
merged 86 commits into from
Apr 3, 2024
Merged

feat(Gadget/prism): 新的代码高亮工具 #443

merged 86 commits into from
Apr 3, 2024

Conversation

AnnAngela
Copy link
Member

@bhsd-harry 佬,看看这个,能不能把你那个parser整合进这里来

如果可以整合的话,先把以下内容删掉:

- type: npm
gadget:
name: prism-language-wiki
fileName: MediaWiki:Gadget-prism-language-wiki.js
moduleName: prismjs
distFilePath: components/prism-wiki.js
version: "1"

然后把这个文件删掉:https://github.com/MoegirlPediaInterfaceAdmins/MoegirlPediaInterfaceCodes/blob/feat/prism/src/gadgets/prism-language-wiki/.eslintrc.yaml

就可以在src/gadgets/prism-language-wiki里写了

AnnAngela and others added 30 commits March 25, 2024 17:51
@bhsd-harry
Copy link
Contributor

我应该fork这个分支吗?或者也可以先合掉,我再提交后续pr。

@AnnAngela
Copy link
Member Author

AnnAngela commented Mar 30, 2024

我应该fork这个分支吗?或者也可以先合掉,我再提交后续pr。

佬你可以直接在这里改,看pr最开头的说明操作就行了

如果要引入站外代码的话最好是参考其他文件用prefetch或者browserify

@bhsd-harry
Copy link
Contributor

佬你可以直接在这里改

我没有这个分支的写入权限。

@AnnAngela
Copy link
Member Author

佬你可以直接在这里改

我没有这个分支的写入权限。

那你fork这个分支提pr吧

@bhsd-harry
Copy link
Contributor

有办法测试吗?还是说先限制一下权限合掉好方便测试?

@lovelyCARDINAL
Copy link
Member

我觉得可以?

@AnnAngela AnnAngela merged commit 2d5d0ae into master Apr 3, 2024
10 checks passed
@AnnAngela AnnAngela deleted the feat/prism branch April 3, 2024 02:08
@AnnAngela AnnAngela restored the feat/prism branch April 3, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants