Skip to content

Conversation

@deads2k
Copy link
Contributor

@deads2k deads2k commented Dec 17, 2015

Fixes formatting (double indent) and destination resolution.

Fixes #6356

@Kargakis @stevekuznetsov

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove any namespaces. There are times I am running these locally and having a namespace is really annoying.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove any namespaces. There are times I am running these locally and having a namespace is really annoying.

done

@0xmichalis
Copy link
Contributor

one comment, LGTM otherwise

@deads2k
Copy link
Contributor Author

deads2k commented Dec 18, 2015

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4419/) (Image: devenv-rhel7_2997)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to e437530

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to e437530

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/7971/)

openshift-bot pushed a commit that referenced this pull request Dec 19, 2015
@openshift-bot openshift-bot merged commit b2568ba into openshift:master Dec 19, 2015
@deads2k deads2k deleted the fix-formatting branch February 26, 2016 18:43
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.

3 participants