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

Don't touch user's configuration file #14

Merged
merged 4 commits into from
Jan 5, 2016
Merged

Don't touch user's configuration file #14

merged 4 commits into from
Jan 5, 2016

Conversation

lord63
Copy link
Owner

@lord63 lord63 commented Jan 4, 2016

#12 Now we won't touch users' configuration file. But the tests still sucks, too many mocks... Orz

@codecov-io
Copy link

Current coverage is 100.00%

Merging #14 into master will not affect coverage as of abedde8

@@            master     #14   diff @@
======================================
  Files            4       4       
  Stmts          110     110       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            110     110       
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of abedde8

Powered by Codecov. Updated on successful CI builds.

@lord63 lord63 merged commit 4fd98c2 into master Jan 5, 2016
@lord63 lord63 deleted the improve-tests branch January 5, 2016 02:19
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.

2 participants