Skip to content

Conversation

@0xmichalis
Copy link
Contributor

No description provided.

@smarterclayton
Copy link
Contributor

How'd this get in without failing the build?

@smarterclayton
Copy link
Contributor

LGTM [merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/1738/)

@openshift-bot
Copy link
Contributor

Evaluated for origin up to aa5cc42

@ncdc
Copy link
Contributor

ncdc commented Apr 30, 2015

gofmt hasn't been running since go1.4 RPMs became available. There's at least 1 PR to fix that (#1070)

@smarterclayton
Copy link
Contributor

Very irritating.

smarterclayton added a commit that referenced this pull request Apr 30, 2015
@smarterclayton smarterclayton merged commit 46ffe7a into openshift:master Apr 30, 2015
@0xmichalis 0xmichalis deleted the gofmt-fix branch May 4, 2015 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants