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

Extend the include syntax #391

Merged
merged 1 commit into from
Feb 4, 2021
Merged

Extend the include syntax #391

merged 1 commit into from
Feb 4, 2021

Conversation

wevsty
Copy link
Contributor

@wevsty wevsty commented Jan 30, 2021

针对include语法进行扩展
详情请见 issues #390

@IceCodeNew
Copy link
Collaborator

这段代码修改在我看来没什么问题,但因为我在 golang 上的经验甚少,之前也没给这个项目贡献过 go 代码,我认为交给 @Loyalsoldier 来排版比较合适。
另外我在思考 @Loyalsoldier 可否重开 PR #255,应该可以配合这次的 PR。

@Loyalsoldier
Copy link
Collaborator

先合并。但需要提及的是,本 PR 目前的一些限制和不足,请看 #390

@Loyalsoldier Loyalsoldier merged commit e640ac2 into v2fly:master Feb 4, 2021
github-actions bot added a commit to YW5vbnltb3Vz/domain-list-community that referenced this pull request Feb 4, 2021
Loyalsoldier added a commit to Loyalsoldier/domain-list-custom that referenced this pull request Apr 2, 2021
Loyalsoldier added a commit to Loyalsoldier/domain-list-custom that referenced this pull request Apr 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants