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

JSHints CanJS #707

Merged
merged 5 commits into from Jan 30, 2014

Conversation

Projects
None yet
2 participants
@justinbmeyer
Contributor

justinbmeyer commented Jan 29, 2014

Adds JSHint error detection to tests and includes corrected errors in the code base.

Stan Carrico and others added some commits Jan 20, 2014

processing jshint modifications to codebase, includes jshint integrat…
…ion with Gruntfile test task, adds new quality taskthat includes jsbeautify
Merge branch 'master' into jshint
Conflicts:
	component/component.js
	component/component_test.js
	control/route/route.js
	control/route/route_test.js
	route/pushstate/pushstate.js
	util/destroyed.js
	util/zepto/zepto.js
	util/zepto/zepto_test.js
@justinbmeyer

This comment has been minimized.

Show comment
Hide comment
@justinbmeyer

justinbmeyer Jan 29, 2014

Contributor

using == here is correct.

Contributor

justinbmeyer commented on util/fixture/fixture.js in 5f189bd Jan 29, 2014

using == here is correct.

@justinbmeyer justinbmeyer merged commit f787605 into master Jan 30, 2014

1 check passed

default The Travis CI build passed
Details

@daffl daffl deleted the jshint branch Feb 24, 2014

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