Skip to content
This repository was archived by the owner on Mar 13, 2020. It is now read-only.

[OSC-1304] - log summary fixes #36

Merged
merged 1 commit into from
Apr 10, 2019
Merged

[OSC-1304] - log summary fixes #36

merged 1 commit into from
Apr 10, 2019

Conversation

@ChintanRaval
Copy link
Collaborator

@ChintanRaval ChintanRaval commented Apr 9, 2019 •
edited
Loading

  • The current multi-line log message for the execution summary is difficult to search for in logs, so switching back to single-line summary message
  • The current unformatted numbers in log are hard to read, so adding a thousand separator to improve read-ablity

@ChintanRaval ChintanRaval requested a review from a team April 9, 2019 23:31
@ChintanRaval ChintanRaval merged commit 3c050e2 into master Apr 10, 2019
@ChintanRaval ChintanRaval deleted the feature/OSC-1304-log-summary-fixes branch April 10, 2019 00:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@waltaro waltaro waltaro approved these changes

@seanbudd seanbudd seanbudd approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ChintanRaval @waltaro @seanbudd