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

feat: support ignore-scripts #59

Merged
merged 3 commits into from
Mar 28, 2016
Merged

feat: support ignore-scripts #59

merged 3 commits into from
Mar 28, 2016

Conversation

dead-horse
Copy link
Member

closes #58

@dead-horse dead-horse added this to the 1.x milestone Mar 28, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @fengmk2 to be a potential reviewer

@fengmk2
Copy link
Member

fengmk2 commented Mar 28, 2016

+1

@codecov-io
Copy link

Current coverage is 93.59%

Merging #59 into master will decrease coverage by -0.58% as of a7eca53

@@            master     #59   diff @@
======================================
  Files           18      18       
  Stmts          790     796     +6
  Branches       139     142     +3
  Methods          0       0       
======================================
+ Hit            744     745     +1
  Partial          0       0       
- Missed          46      51     +5

Review entire Coverage Diff as of a7eca53

Powered by Codecov. Updated on successful CI builds.

@fengmk2
Copy link
Member

fengmk2 commented Mar 28, 2016

+1

@fengmk2 fengmk2 merged commit 40af7e6 into master Mar 28, 2016
@fengmk2 fengmk2 deleted the ignore-scripts branch March 28, 2016 08:29
@fengmk2
Copy link
Member

fengmk2 commented Mar 28, 2016

1.5.0

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

Successfully merging this pull request may close these issues.

support --ignore-scripts
4 participants