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

Improved unittests #143

Merged
merged 5 commits into from
Jan 18, 2016
Merged

Conversation

evilaliv3
Copy link
Contributor

@ericclemmons @underscorebrody, i noticed that the dev dependencies where really outdated.

i took care of retesting the new stables and to update the only one test that required a change in order to be aligned to latest usemin.

in addition i've applied various improvements:

  • unitetests have been switched to travis container based infrastructure (that would allow a minor speed up of builds)
  • npm packages are now subject to travis caching e (that would allow a minor speed up of builds and less networking overhead for travis)
  • tests are now performed also on on node 4.2 (that is now the LTS and so the most important version to test against)

@underscorebrody
Copy link
Collaborator

This seems good to me, my only concern is that a couple of dependencies are going up full point releases. Will probably need to make this release a full point as well to be safe. Tests are passing though so it looks pretty safe to me 👍

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