Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CLI refactoring #954

Merged
merged 1 commit into from
Jul 9, 2018
Merged

Conversation

cryptocode
Copy link
Contributor

@cryptocode cryptocode commented Jul 7, 2018

  • Moves CLI commands into a separate source file, reducing node.cpp by 700+ lines.
  • Use std::error_code instead of bool returns. This solves a problem with the help listing being displayed after errors, obscuring any error messages.

@clemahieu clemahieu merged commit ef86a5e into nanocurrency:master Jul 9, 2018
@cryptocode cryptocode deleted the cli-refactoring branch July 9, 2018 23:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants