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)

Adding --locked to explorer.Dockerfile as D17313 did not specify cargo build --release --locked

Changed cargo build to cargo build --locked to ensure reproducible builds

Updated lint

Updated formatting

testing to see if it was faulty test

test

testing

Updated every dependancy in Cargo.toml that does not require any change in code.

Changed bitcoin_hashes version back to 0.14

updated indents

Updated formatting

added new line \ so build should work