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

Print integration test output on a newline #598

Merged
merged 1 commit into from
May 18, 2017
Merged

Print integration test output on a newline #598

merged 1 commit into from
May 18, 2017

Conversation

carolynvs
Copy link
Collaborator

This makes is easier to quickly scan the test output, without messing up the section header indents.

Here's a sample of how that looks:

integration_testproj.go:180: standard error:
        		Finding dependencies of "github.com/golang/notexist"...
        		  Found 2 dependencies.
        		Detected glide configuration files...
        		  Loading /private/var/folders/_t/fbw4wf_s42dfqdfjq7shdfm40000gn/T/gotest490168315/src/github.com/golang/notexist/glide.yaml
        		  Loading /private/var/folders/_t/fbw4wf_s42dfqdfjq7shdfm40000gn/T/gotest490168315/src/github.com/golang/notexist/glide.lock
        		Converting from glide.yaml and glide.lock...

@carolynvs
Print integration test output on a newline
b373c4f 
This makes is easier to quickly scan the test output, without messing up the section header indents
jmank88
jmank88 approved these changes May 17, 2017
View reviewed changes
Copy link
Collaborator

@jmank88 jmank88 left a comment

Choose a reason for hiding this comment

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

LGTM

@sdboyer
Copy link
Member

sdboyer commented May 18, 2017

good call good call

@sdboyer sdboyer merged commit 2879206 into golang:master May 18, 2017
@carolynvs carolynvs deleted the print-testoutput-newline branch May 18, 2017 01:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jmank88 jmank88 jmank88 approved these changes

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@carolynvs @sdboyer @jmank88 @googlebot