Skip to content

[WIP] Adding build script#44

Open
wachterjohannes wants to merge 2 commits into
puli:masterfrom
wachterjohannes:master
Open

[WIP] Adding build script#44
wachterjohannes wants to merge 2 commits into
puli:masterfrom
wachterjohannes:master

Conversation

@wachterjohannes

@wachterjohannes wachterjohannes commented Sep 24, 2016

Copy link
Copy Markdown

This PR will add an automatic build with travis.

This travis-builder will be used to test the deployment: https://travis-ci.org/wachterjohannes/cli

@wachterjohannes

wachterjohannes commented Sep 24, 2016

Copy link
Copy Markdown
Author

@webmozart do have you an idea why the tests failing on my own travis channel?

@wachterjohannes

Copy link
Copy Markdown
Author

we also have to declare where you store the build foreach version. we should chat maybe in the sulu slack-channel about the details. i think i have build the basics here and only have to adapt it when the details are defined.

Comment thread .travis.yml
directories:
- $HOME/.composer/cache/files

- $HOME/.composer/cache/files

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there any reason why you removed these spaces?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

no not really but the travis shell script removes it (:

Comment thread bin/deploy.sh
chmod 755 box.phar
./box.phar build -vv

# TODO if build with gh-pages

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's happening with these TODOs?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

i will remove it when its clear how we deploy the files (:

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.

2 participants