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

Fix: Added timeout configuration to NodeClient, both bwallet-cli and bcoin-cli #1132

Merged
merged 1 commit into from
May 11, 2023

Conversation

manavdesai27
Copy link
Contributor

Fixes #967 .

Made changes as proposed in the solution, suggested in the issue.

@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (57f5956) 69.53% compared to head (6f9ac5c) 69.52%.

❗ Current head 6f9ac5c differs from pull request most recent head 4092f36. Consider uploading reports for the commit 4092f36 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1132      +/-   ##
==========================================
- Coverage   69.53%   69.52%   -0.02%     
==========================================
  Files         158      158              
  Lines       26598    26598              
==========================================
- Hits        18494    18491       -3     
- Misses       8104     8107       +3     

see 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@masterchief164
Copy link
Collaborator

LGTM! Just squash the commits and we are good to go!

@manavdesai27
Copy link
Contributor Author

LGTM! Just squash the commits and we are good to go!

Done!

Copy link
Collaborator

@masterchief164 masterchief164 left a comment

Choose a reason for hiding this comment

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

Ack!

@pinheadmz pinheadmz merged commit 9b45034 into bcoin-org:master May 11, 2023
# 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.

Can not adjust client timeout option within the CLI
4 participants