Skip to content
This repository has been archived by the owner on Feb 26, 2021. It is now read-only.

文章贡献指南 #51

Merged
merged 2 commits into from
Jun 25, 2019

Conversation

loverto
Copy link
Contributor

@loverto loverto commented Jun 20, 2019

请审校者在合并该 PR 前需要检查以下内容:

  • Header 设置正确,包括文章的类型、tag、分类、作者、译者、摘要、reviewer 发布时间等信息
  • 关联 Issue 编号 ([feature] 添加文章收录规范 #20)
  • 该 PR Assigned 给了提交者
  • 指定了 Reviewer
  • 为 PR 明确了 label 信息,如 categories、kind、size、tag、status
  • 所有代码片段都指定了代码语言
  • 翻译的文章添加了编者按
  • 文章遵守了路径规范 YEAR/MONTH/TITLE/index.md,例如 2019/04/new-blog/index.md
  • 图片保存了在 GitHub上,跟文章在同级目录
  • 文章标题简短且语义明确
  • 文章语句通顺,没有语病和格式问题
  • 文中没有不可达的链接
  • 提交者同意授权 ServiceMesher 公众号和 http://www.servicemesher.com 发布我原创或翻译的文章

当所有以上问题都确定之后,可以由管理者合并 PR 并关闭关联 issue。

@auto-comment
Copy link

auto-comment bot commented Jun 20, 2019

Thank you for raising the pull request. We will review it as soon as possible.
To ensure document quality, we have enabled the CI test, which currently includes spell and format checks. If your pull request fails to pass the spell check, you can check circleci find more detail to fix it. 😆

@auto-assign auto-assign bot requested a review from x19990416 June 20, 2019 09:32
CONTRIBUTING.md Outdated Show resolved Hide resolved
@GuangmingLuo GuangmingLuo merged commit f8507be into servicemesher:master Jun 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants