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.

Sign up for GitHub

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

Jump to bottom

Update JSHint settings / Update package.json #336

Merged
merged 3 commits into from
Nov 22, 2014
Merged

Update JSHint settings / Update package.json #336

merged 3 commits into from
Nov 22, 2014

Conversation

shinnn
Copy link
Contributor

@shinnn shinnn commented Nov 22, 2014

  • I enabled strict option of JSHint.
    • Almost all files use strict mode.
  • I added bin/browsersync to the target file list of lint task.
  • I added files field to package.json and removed .npmignore.
  • I updated dependencies and devDependencies.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 1fde0c4 on shinnn:patch-1 into c1b820b on shakyShane:master.

@shakyShane
Copy link
Contributor

Nice work, as always!

Very much appreciated :)

shakyShane added a commit that referenced this pull request Nov 22, 2014
@shakyShane
Merge pull request #336 from shinnn/patch-1
58cd108 
Update JSHint settings / Update package.json
@shakyShane shakyShane merged commit 58cd108 into BrowserSync:master Nov 22, 2014
@shinnn shinnn deleted the patch-1 branch November 22, 2014 09:56
@shinnn
Copy link
Contributor Author

shinnn commented Nov 22, 2014

No probs ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

None yet
3 participants
@shinnn @coveralls @shakyShane