Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable travis ci builds #3

Closed
wants to merge 20 commits into from
Closed

Enable travis ci builds #3

wants to merge 20 commits into from

Conversation

lb1a
Copy link
Contributor

@lb1a lb1a commented Jul 10, 2017

I've added the travis.yml needed for automated testing with travis.
I've used the latest stable nodejs and the latest stable node js LTS version als build targets, with the custom build command.
You can see the output here: https://travis-ci.org/lb1a/ole2
And of course they offer nice badges 😄
Build Status

@lb1c
Copy link
Contributor

lb1c commented Jul 12, 2017

Travis Deployment does not handle files ignored by .gitignore. So the page build will always be missing the generated/downloaded files in dist and node_modules.
Problem documented here: travis-ci/travis-ci#4006
Workaround is either removing the folders from gitignore or using a custom deploy script:
https://github.com/felixrieseberg/travis-jekyll-git

@lb1a
Copy link
Contributor Author

lb1a commented Jul 13, 2017

@mhaertwig
The build is working, but pushing to gh-pages with the node_modules folder fails, because the deps include an *.exe file.
This gets blocked by github. Could you please clarify which files are actually needed in the deployment?

The page build completed successfully, but returned the following warning for the gh-pages branch:

It looks like you're using GitHub Pages to distribute binary files. We strongly suggest that you use releases to ship projects on GitHub. Releases are GitHub's way of packaging and providing software to your users. You can think of it as a replacement to using downloads to provide software. We found the following file(s) > which may be a good candidate for releases: node_modules/closure-util/.deps/library/0f95ad8150b77cd8db0e8d9e9726292ab33580af/scripts/ci/CloseAdobeDialog.exe. For more information, see https://help.github.com/articles/about-releases/.

For information on troubleshooting Jekyll see:

https://help.github.com/articles/troubleshooting-jekyll-builds

@lb1c
Copy link
Contributor

lb1c commented Jul 14, 2017

Closing messy PR, recreating cleaned PR

@lb1c lb1c closed this Jul 14, 2017
@lb1a lb1a deleted the enable-travis-ci branch July 14, 2017 21:38
lb1c added a commit that referenced this pull request Feb 19, 2018
Update webpack to the latest version 🚀
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants