Linter

fixed error

Downcast the Report into ChronikClientError, so that we can now match on status code being returned (thanks Fabien!). Updated variable names from load to get, updated panic messages

rebase

Updated comment

Updating comment (linter plz dont kill me )

Updated !unreachable() comment and removed assert comments

Merged imports

Updated the assert!() so that it will now anticipate 404 errors, renamed function names to be explicit

Rename blockchain_info_ipc

lint

lint

Added unreachable panic, renamed test_blockchain_info to blockchain_info and added comments

Renamed blockchain_info_new to blockchain_info

sorry relint

lint

I will make the changes from Pierre's comment in a separate diff to all current integration tests.

Revamped blockchain_info to include the new trait changes.

lint

Asserted at the end of test to make sure we return an error. Killing child process won't return an error, so we get the returned error from assert

Removed timeout

Added error handling

New line added + updated server to make explicit

Removed assert_eq

Removed useless comment

Deleted second test, made socket error handling verbose

Made changes suggested

Added the regex test for version returned from ChronikInfo

Added assert_eq!() test for chronik version returned

Added CI integration (build-configurations.yml), added copyright licenses, changed server and reduced Tokio syntax.

Moved modules/bitcoinsuite-chronik-client next to other modules in workspace. Moved chronik-proto = { path = "../../chronik/chronik-proto/"} to other local dependencies

Updating lint

Deleted newline in hash.rs

Import chronik-proto from /chronik

Updated everything in this diff - please see summary for more indepth

In D17335#393946, @bytesofman wrote:

In D17335#393945, @Fabien wrote:

In D17335#393933, @bytesofman wrote:

nice.

is this a greenfield lib or is it initializing a repo in the monorepo that exists somewhere else?

This is Tobias' client, the one we use for the explorer. @hazzarust is putting it into the monorepo then will add the failover proxy to it.

ok nice

in that case, not really a big deal about adding stuff like test comments, having this as a starting point is helpful

fixed indentation issue (timeout was indented in script, removed indentation)