Skip to content

chore(manifests): remove repository manifests#73

Merged
arschles merged 2 commits into
deis:masterfrom
arschles:rm-manifests
May 25, 2016
Merged

chore(manifests): remove repository manifests#73
arschles merged 2 commits into
deis:masterfrom
arschles:rm-manifests

Conversation

@arschles

@arschles arschles commented Feb 18, 2016

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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