You are viewing a single comment's thread from:

RE: Added new features to Blagominer: support for HTTPS connections and servers requiring custom headers

in #utopian-io5 years ago

Thank you for your contribution. The PR has not been merged and you are forking the original repro. And, for next time, could you also verify your github account by e.g. linking your steem URL in your github profile?

  1. The post has contained many information, it would be good if you can restructure it. You don't have to give too much details on the branches etc. Sections Headlines would be useful.
  2. the original project Blagominer seems unmaintained.
  3. There are some non-English comments in the code-base.
  4. code with deep nested blocks are hard to follow.
  5. no tests

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Chat with us on Discord.

[utopian-moderator]

Sort:  

Thank you for your review, @justyy! Keep up the good work!

Coin Marketplace

STEEM 0.28
TRX 0.13
JST 0.033
BTC 62199.58
ETH 3014.60
USDT 1.00
SBD 3.57