add lgtm config - Githubissues

mipengine / mip2

MIP (移动网页加速器)通过优化网页JS、控制资源加载顺序，达到加速网页的效果。

https://www.mipengine.org/

MIT License

184 stars 49 forks source link

add lgtm config #551

Closed chenqiushi closed 5 years ago

chenqiushi commented 5 years ago

相关 ISSUE: （ISSUE 链接地址）

1、升级点 （清晰准确的描述升级的功能点）

增加 lgtm auto code review 机制

2、影响范围 （描述该需求上线会影响什么功能）

持续继承功能，对核心代码无影响

3、自测 Checklist

4、需要覆盖的场景和 Case

[ ] 是否覆盖了 sf 打开 MIP 页
[ ] 是否验证了极速服务 MIP 页面效果

5、自测机型和浏览器

[ ] 是否覆盖了 iOS 系统手机
[ ] 是否覆盖了 Android 系统手机
[ ] 是否覆盖了 iPhone 版式浏览器（比如 QQ、UC、Chrome、Safari、安卓自带浏览器）
[ ] 是否覆盖了手百

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 995

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.408%

Totals
Change from base Build 992:	0.0%
Covered Lines:	3746
Relevant Lines:	3888

💛 - Coveralls

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 977

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.408%

Totals
Change from base Build 992:	0.0%
Covered Lines:	3746
Relevant Lines:	3888

💛 - Coveralls

mipengine-bot commented 5 years ago

This pull request fixes 11 alerts when merging 7e2cd1145c7edf2b8260c530c946c78d87139679 into 4de31f689362eb63a72278dc4b7b902cbdb1484b - view on LGTM.com

fixed alerts:

4 for Unused variable, import, function or class
2 for Superfluous trailing arguments
2 for Ambiguous HTML id attribute
1 for Client-side cross-site scripting
1 for Conflicting HTML element attributes
1 for Incomplete string escaping or encoding

Comment posted by LGTM.com