Skip to content
This repository was archived by the owner on Feb 19, 2019. It is now read-only.

Conversation

@LeafHacker
Copy link
Contributor

Inspired by the way Forge builds its MDK, we should do the same with out CDK.

We don't really need a separate repo for the CDK, when really we want to build a CDK zip whenever we update ClientAPI.

This commit adds a build task to do just that. It takes generic files from ClientAPI (license, gradle), CDK specific files and the example source code and archives them as a zip file in build/distributions.

This means we kan keep all the CDK specific files here (in a cdk dir) and stop maintaining yet another repo.


This also means that the example code is no longer needed in multiple locations and we will be able to bin the complicated submodule setup. I've not done that here to keep the commit small/single purpose and because the example repo isn't up to date with @ZeroMemes's local copy.

We don't really need a separate repo for the CDK, when really we want to build a CDK zip whenever we update ClientAPI. This commit adds a build task to do just that. It takes generic files from ClientAPI (license, gradle), CDK specific files and the example source code and archives them as a zip file in build/distributions. This means we kan keep all the CDK specific files here (in a cdk dir) and stop maintaining yet another repo.
@LeafHackerLeafHacker added this to the 2.2 milestone Aug 13, 2017
@LeafHackerLeafHacker changed the title Make a CDK from files here instead of having a CDK repoMake a CDK zip instead of having a CDK repoAug 13, 2017
Copy link
Member

@ZeroMemesZeroMemes left a comment

Choose a reason for hiding this comment

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

Looks good™

@LeafHackerLeafHacker merged commit 1731ee5 into ImpactDevelopment:masterAug 14, 2017
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@LeafHacker@ZeroMemes