Skip to content

Conversation

@arschles
Copy link
Member

@arschles arschles commented Feb 18, 2016

In the same vein as deis/postgres#38, prefer to use the helm chart manifest to the repository manifests

Ref deis/builder#188

Still TODO:

  • Remove build targets and Makefile variables that reference local manifests

@krancour
Copy link
Contributor

Shouldn't there be some complementary changes to the Makefile? There are targets that use these manifests and I think they're dead weight now.

@arschles
Copy link
Member Author

@krancour yes, removed parts of the Makefile that reference local manifests in abd2497

@arschles arschles added this to the v2.0 milestone May 23, 2016
@smothiki smothiki added the LGTM2 label May 25, 2016
@arschles arschles merged commit b2196b8 into deis:master May 25, 2016
@arschles arschles deleted the rm-manifests branch May 25, 2016 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants