Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

CLI: Export error message and less verbose peer counter. #5870

Merged
merged 3 commits into from
Jun 27, 2017
Merged

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Jun 19, 2017

No description provided.

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Jun 19, 2017
@arkpar arkpar changed the title CLI: Error export message and less verbose peer counter. CLI: Export error message and less verbose peer counter. Jun 19, 2017
@rphmeier rphmeier added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 19, 2017
@tomusdrw tomusdrw added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A8-looksgood 🦄 Pull request is reviewed well. labels Jun 19, 2017
@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jun 19, 2017
@tomusdrw tomusdrw requested a review from jacogr June 19, 2017 12:57
@NikVolf NikVolf added A8-looksgood 🦄 Pull request is reviewed well. A0-pleasereview 🤓 Pull request needs code review. and removed A0-pleasereview 🤓 Pull request needs code review. A8-looksgood 🦄 Pull request is reviewed well. labels Jun 20, 2017
Copy link
Contributor

@jacogr jacogr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, simplifies.

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 27, 2017
@arkpar arkpar merged commit 4a7bcfe into master Jun 27, 2017
@arkpar arkpar deleted the cli-fixes branch June 27, 2017 20:40
arkpar added a commit that referenced this pull request Jul 14, 2017
* Removed numbed of active connections from informant

* Print error message when fatdb is required

* Remove peers from UI
arkpar added a commit that referenced this pull request Jul 14, 2017
* --reseal-on-uncle (#5940)

* --reseal-on-uncle

* Optimized uncle check

* Additional uncle check

* Updated comment

* v1.6.9

* CLI: Export error message and less verbose peer counter. (#5870)

* Removed numbed of active connections from informant

* Print error message when fatdb is required

* Remove peers from UI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants