Skip to content

Fix issue with test result logs#3246

Merged
alexperovich merged 1 commit intodotnet:masterfrom
alexperovich:logs
Jul 9, 2019
Merged

Fix issue with test result logs#3246
alexperovich merged 1 commit intodotnet:masterfrom
alexperovich:logs

Conversation

@alexperovich
Copy link
Copy Markdown
Member

@alexperovich alexperovich commented Jul 9, 2019

indentation.......

@alexperovich alexperovich self-assigned this Jul 9, 2019
@alexperovich alexperovich requested a review from ChadNedzlek July 9, 2019 01:16
MattGal
MattGal approved these changes Jul 9, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@MattGal MattGal left a comment

Choose a reason for hiding this comment

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

LGTM

src/Microsoft.DotNet.Helix/Sdk/tools/azure-pipelines/reporter/test_results_reader/__init__.py
text=log_list,
))
total_added_logs += 1
total_added_logs += 1
Copy link
Copy Markdown
Member

@MattGal MattGal Jul 9, 2019

Choose a reason for hiding this comment

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

subtle!

@alexperovich alexperovich merged commit 71581f0 into dotnet:master Jul 9, 2019
@alexperovich alexperovich deleted the logs branch July 9, 2019 16:45
This was referenced Jul 28, 2022
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MattGal MattGal MattGal approved these changes

@missymessa missymessa missymessa approved these changes

@ChadNedzlek ChadNedzlek Awaiting requested review from ChadNedzlek

Assignees

@alexperovich alexperovich

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexperovich @MattGal @missymessa