6.08 上线 - Githubissues

mipengine / mip2

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

https://www.mipengine.org/

MIT License

184 stars 49 forks source link

6.08 上线 #671

Closed clark-t closed 5 years ago

clark-t commented 5 years ago

相关 ISSUE: （ISSUE 链接地址）

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

MIP on/bind 机制升级
修复 mip-img popup bug

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 1365

0 of 0 changed or added relevant lines in 0 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 94.473%

Files with Coverage Reduction	New Missed Lines	%
src/components/mip-carousel.js	1	99.16%
src/mip1-polyfill/customElement.js	1	96.67%
src/util/dom/dom.js	1	96.23%
<!--	Total:	3	-->

Totals
Change from base Build 1317:	0.2%
Covered Lines:	4345
Relevant Lines:	4489

💛 - Coveralls

mipengine-bot commented 5 years ago

This pull request fixes 1 alert when merging a0f2045fbd545affa5ec1951e2b1884b47ad46bc into 6d11c924d39dfb236c84f5ff2a690d412805b9bb - view on LGTM.com

fixed alerts:

1 for Useless assignment to local variable

This project has automated code review enabled, but doesn't use the LGTM GitHub App. Migrate over by installing the app. Read about the benefits of migrating to GitHub Apps in the blog.

Comment posted by LGTM.com