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

[modify]: improve .travis file. Use a recursive way to check Markdown links #601

Merged
merged 1 commit into from
Apr 26, 2018

Conversation

zihengCat
Copy link
Contributor

修改项

  • 完善.travis.yml,项目持续集成(CI)功能应该可以正常使用了。
  • 改进check_links.sh脚本文件,使用递归方法遍历,不会遗漏.md文件了。

建议

README.md中的一些外链已挂,建议补链。(参考下图)
沈阳事件.md中大部分参考链接都是无效的,.md文档结构也很乱,建议完善。
screen shot 2018-04-26 at 1 38 49

@gaocegege gaocegege added the lgtm label Apr 26, 2018
@gaocegege
Copy link

We need to enable Travis CI for the repository.

@vzxxbacq
Copy link
Collaborator

vzxxbacq commented Apr 26, 2018

@gaocegege README中有build status,应该已经开启了。

@gaocegege
Copy link

https://travis-ci.org/sikaozhe1997/Xin-Yue

It seems that it is not activated in Travis.

@juliosueiras
Copy link
Contributor

Ref: #557

@sikaozhe1997 sikaozhe1997 merged commit cfcfdbb into sikaozhe1997:master Apr 26, 2018
@vzxxbacq
Copy link
Collaborator

@sikaozhe1997 需要您去travis开启这个功能。
可参考https://www.liaoxuefeng.com/article/0014631488240837e3633d3d180476cb684ba7c10fda6f6000

@zihengCat
Copy link
Contributor Author

@sikaozhe1997
麻烦您去 Travis CI 上点一下呗。


screen shot 2018-04-26 at 12 33 08

@zihengCat
Copy link
Contributor Author

zihengCat commented Apr 26, 2018

@sikaozhe1997 @gaocegege @vzxxbacq

Travis CI 已启用,只要.md文章中存在死链,测试便无法通过(build:failed)。
可前往 Travis CI 项目Build Jobs -> Job Log查看死链并作相应调整。(例:https://travis-ci.org/sikaozhe1997/Xin-Yue/jobs/371384425

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants