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

fix test #115

Merged
merged 2 commits into from Oct 21, 2016
Merged

fix test #115

merged 2 commits into from Oct 21, 2016

Conversation

helloyou2012
Copy link
Contributor

@helloyou2012 helloyou2012 commented Oct 21, 2016

fix #27

@mention-bot
Copy link

@helloyou2012, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fengmk2 to be a potential reviewer.

@helloyou2012
Copy link
Contributor Author

appveyor 的 CI 请求超时了 😓

@codecov-io
Copy link

Current coverage is 89.34% (diff: 100%)

Merging #115 into master will decrease coverage by 0.62%

@@             master       #115   diff @@
==========================================
  Files             8          8          
  Lines           349        338    -11   
  Methods          43         43          
  Messages          0          0          
  Branches        100         88    -12   
==========================================
- Hits            314        302    -12   
- Misses           35         36     +1   
  Partials          0          0          

Powered by Codecov. Last update c121038...6b50ee9

@fengmk2
Copy link
Member

fengmk2 commented Oct 21, 2016

@helloyou2012 很棒!ci 过了就合并。

@fengmk2
Copy link
Member

fengmk2 commented Oct 21, 2016

我看只是2个 task 并行冲突的问题,没问题了,我合并一下。

@fengmk2
Copy link
Member

fengmk2 commented Oct 21, 2016

+1

@fengmk2 fengmk2 merged commit ce34145 into node-modules:master Oct 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

测试一直挂着。
4 participants