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.

Sign up for GitHub

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

Jump to bottom

update cdn list #314

Closed
wants to merge 10 commits into from
Closed

update cdn list #314

wants to merge 10 commits into from

Conversation

wevsty
Copy link
Contributor

@wevsty wevsty commented Dec 12, 2020

No description provided.

@IceCodeNew IceCodeNew self-requested a review December 13, 2020 08:24
@wevsty
update wangsu.
d327c73 
add cn attr for wangsu.
@kslr
Copy link
Contributor

kslr commented Jan 5, 2021

如果 review 受到了阻力，我建议你拆分更小的 commit 或者有帮助快速检查数据来源的办法

@wevsty
Copy link
Contributor Author

wevsty commented Jan 5, 2021

如果 review 受到了阻力，我建议你拆分更小的 commit 或者有帮助快速检查数据来源的办法

目前的话，我想先等一等 issues #28 最后的建议来实装，后续可能还需要调整。

@IceCodeNew
Copy link
Collaborator

IceCodeNew commented Jan 5, 2021 •
edited
Loading

等我 17 号考完试我可以来详细看看这个 PR，刚好我和 contributor 联系起来也方便。

@wevsty
Copy link
Contributor Author

wevsty commented Feb 5, 2021

由于新语法的支持，我需要重新整理，所以先关闭这个PR。
后续我会拆成更小的PR再提交。

@wevsty wevsty closed this Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IceCodeNew IceCodeNew Awaiting requested review from IceCodeNew

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wevsty @kslr @IceCodeNew