search

hyperledger / indy-node-monitor

Apache License 2.0
13 stars 28 forks source link

Fix for missing linux packages and docker-composer check #65

Closed crajapakshe closed 1 year ago

crajapakshe commented 1 year ago

Some small recommendations below. Could you also squash all the changes into a single commit to clean up the history please?

@WadeBarnes Updated.

WadeBarnes commented 1 year ago

My latest review comments make more sense in context as comments on the preview review comments where I actually made the suggested changes. It seems GitHub has yanked them out of context in the most recent review.

crajapakshe commented 1 year ago

My latest review comments make more sense in context as comments on the preview review comments where I actually made the suggested changes. It seems GitHub has yanked them out of context in the most recent review.

Thanks for pointed out and update the code with suggestions.