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

Feature/a bit of ci #17

Closed
wants to merge 5 commits into from
Closed

Conversation

@mfeckie
Copy link
Contributor

mfeckie commented Nov 19, 2015

Resolves #15

TODO - Owner of repo / person merging needs to enable watching of the repo by Travis CI

Adds support for CI with .travis.yml
Adds package.json to manage JS dependencies
Adds JS Hint, and commands to execute it
Removes some unused files
Switches the JS libraries to all be served by CDN to be kind to users on bandwidth

@mfeckie
Copy link
Contributor Author

mfeckie commented Nov 19, 2015

And that didn't work either! Looks like it's gonna take someone pushing directly to the repo the first time to get Travis to start watching ...

@Ms2ger Ms2ger closed this Nov 19, 2015
@Ms2ger Ms2ger reopened this Nov 19, 2015
@Ms2ger
Copy link

Ms2ger commented Nov 19, 2015

Try adding the change from #19

@mfeckie mfeckie closed this Nov 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.