search

servicemesher / weekly

ServiceMesher Weekly
Apache License 2.0
262 stars 69 forks source link

Add some basic ci scripts #260

Closed SataQiu closed 5 years ago

SataQiu commented 5 years ago

参考 istio.io 项目，添加一些基本自动化测试脚本，以后再逐步完善功能。 /cc @haiker2011 @rootsongjc

auto-comment[bot] commented 5 years ago

Thank your for raising your pull request. We will review it as soon as possible. :laughing:

rootsongjc commented 5 years ago

好像没有加上 404 链接检测？

SataQiu commented 5 years ago

现在脚本里这两个检测是都过不去的，还得抽时间刷一遍格式，另外整理一下词汇表。 目前只是先把脚本放上来了。 而且可能以后还得搞一个测试专用的镜像，以保证测试环境的一致性。 404 检测，我看 istio.io 是先编译成 html，然后使用 htmlproofer 工具检查生成的 html 文件。 咱们的网站打算是也编译后检查，还是直接检查 markdown 文件？现在不清楚怎么做。

haiker2011 commented 5 years ago

对哪些东西做了检查应该形成一个文档，也方便提PR之前可以先提前检查是不是合格，或者告知怎么在提交之前进行预提交检查

SataQiu commented 5 years ago

如果做成自动检查其实文档的重要性也就不那么强了，提 PR CI 失败就知道怎么回事了。 circleci travisci 这类可以考虑下，咱们要不要引入？ 我看很多也用这个来自动执行自定义脚本检查PR的。

haiker2011 commented 5 years ago

@SataQiu 我觉得可以circleci travisci ，配合上脚本应该挺好

rootsongjc commented 5 years ago

同时把 CI 也加进来吧，以验证脚本是否可行。