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 bug related to config overwrite #97

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Fix bug related to config overwrite #97

merged 1 commit into from
Feb 29, 2016

Conversation

vrde
Copy link
Contributor

@vrde vrde commented Feb 29, 2016

Calling bigchaindb configure would overwrite the original config even if the user answered n.

This PR is to fix this wrong behavior.

@r-marques
Copy link
Contributor

👍

@codecov-io
Copy link

Current coverage is 83.29%

Merging #97 into develop will increase coverage by +83.29% as of ca54948

@@            develop     #97   diff @@
=======================================
  Files             0      15    +15
  Stmts             0     778   +778
  Branches          0       0       
  Methods           0       0       
=======================================
+ Hit               0     648   +648
  Partial           0       0       
- Missed            0     130   +130

Review entire Coverage Diff as of ca54948

Powered by Codecov. Updated on successful CI builds.

vrde added a commit that referenced this pull request Feb 29, 2016
Fix bug related to config overwrite
@vrde vrde merged commit 0ed5341 into develop Feb 29, 2016
@r-marques r-marques added this to the v0.2.0 milestone Mar 9, 2016
@vrde vrde deleted the fix-config-overwrite branch March 21, 2018 15:03
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.

3 participants