Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

Improved logging #386

Merged
merged 1 commit into from
Jan 22, 2019
Merged

Improved logging #386

merged 1 commit into from
Jan 22, 2019

Conversation

Jakeeyturner
Copy link
Contributor

First pass at improving logging, however more work needs to be done to make logging consistent.

Signed-off-by: Jake Turner jaketurner25@live.com

client/src/logging/OutputAdapter.ts Outdated Show resolved Hide resolved
client/src/logging/VSCodeOutputAdapter.ts Outdated Show resolved Hide resolved
client/src/util/CommandUtil.ts Outdated Show resolved Hide resolved
client/src/fabric/FabricRuntime.ts Outdated Show resolved Hide resolved
client/src/fabric/FabricConnectionManager.ts Outdated Show resolved Hide resolved
client/src/fabric/FabricConnection.ts Outdated Show resolved Hide resolved
client/src/fabric/FabricClientConnection.ts Outdated Show resolved Hide resolved
@Jakeeyturner Jakeeyturner force-pushed the issue-176 branch 2 times, most recently from 224f057 to 8e3bd4b Compare November 30, 2018 10:52
@Jakeeyturner Jakeeyturner force-pushed the issue-176 branch 5 times, most recently from 5776d0d to ffc6957 Compare January 22, 2019 12:45
cazfletch
cazfletch previously approved these changes Jan 22, 2019
Copy link
Contributor

@cazfletch cazfletch left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Jake Turner <jaketurner25@live.com>
Copy link
Contributor

@cazfletch cazfletch left a comment

Choose a reason for hiding this comment

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

LGTM

@cazfletch cazfletch merged commit cbd649d into IBM-Blockchain:master Jan 22, 2019
This was referenced Jan 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants