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

Support Offline Testing #600

Merged
merged 1 commit into from Dec 10, 2013

Conversation

Projects
None yet
2 participants
@daffl
Contributor

daffl commented Dec 9, 2013

We should support offline testing / development (considering how much I travel). Can we have qunit locally again?

@ghost ghost assigned daffl Dec 8, 2013

@daffl

This comment has been minimized.

Show comment
Hide comment
@daffl

daffl Dec 9, 2013

Contributor

That makes sense. Could we use Bower at least for QUnit and just make it a post-npm-install script? Once installed it will work offline but we still wouldn't need to check it into the main repository.

Contributor

daffl commented Dec 9, 2013

That makes sense. Could we use Bower at least for QUnit and just make it a post-npm-install script? Once installed it will work offline but we still wouldn't need to check it into the main repository.

daffl added a commit that referenced this pull request Dec 10, 2013

@daffl daffl merged commit 3812d50 into master Dec 10, 2013

1 check passed

default The Travis CI build passed
Details

@daffl daffl deleted the offline-testing-600 branch Dec 10, 2013

@daffl

This comment has been minimized.

Show comment
Hide comment
@daffl

daffl Dec 10, 2013

Contributor

Merged into master. Recommending to remove node_modules and reinstall to get everything up to date.

Contributor

daffl commented Dec 10, 2013

Merged into master. Recommending to remove node_modules and reinstall to get everything up to date.

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Dec 18, 2013

Hey could you please change that to "node node_modules/bower/bin/bower install"?
The current version does break npm install under windows.

ghost commented on package.json in 5531be2 Dec 18, 2013

Hey could you please change that to "node node_modules/bower/bin/bower install"?
The current version does break npm install under windows.

This comment has been minimized.

Show comment
Hide comment
@daffl

daffl Dec 18, 2013

Contributor

Sure, just updated master with the change.

Contributor

daffl replied Dec 18, 2013

Sure, just updated master with the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment