Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Add JSHinting and EditorConfig #205

Merged
merged 11 commits into from
Jan 3, 2014
Merged

Add JSHinting and EditorConfig #205

merged 11 commits into from
Jan 3, 2014

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Jan 3, 2014

  • Add http://EditorConfig.org setting file for supported IDEs to use consistent indenting spacing
  • Standardize indenting in JS files
  • Add JSHint to devDependencies and run before npm test task

nschonni added 11 commits January 2, 2014 23:09
@nschonni
Add EditorConfig for indentation standardization
5ba895d
@nschonni
Don't lint node_modules
956dfe9
@nschonni
JSHint Middleware example
c9d9294
@nschonni
JSHint: Add Mocha globals and allow multistr
c0fc5db
@nschonni
Remove outdated test.js
a35b2df
@nschonni
JSHint sass.js
f14b040 
- Fix indenting
- Remove unused callback
@nschonni
JSHint Build.js
43cff35 
- Tabs to spaces
- Remove 'runner' Mocha var since it is only invoked
@nschonni
JSHint source_comments_spec.js test
6d530ba 
Remove unused vars
@nschonni
JSHint test.js
f066eb6 
- Add missing semicolons
- Remove unused parameters
@nschonni
JSHint cli.js tests
b7574c9 
- Remove unused vars
- Remove unused parameters
@nschonni
Run JSHint when testing
bf2e929
nschonni added a commit that referenced this pull request Jan 3, 2014
@nschonni
Merge pull request #205 from nschonni/jshinting
10018d9 
Add JSHinting and EditorConfig
@nschonni nschonni merged commit 10018d9 into sass:master Jan 3, 2014
@nschonni nschonni deleted the jshinting branch January 3, 2014 05:30
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nschonni