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

Add .jshintrc config file to be compliant with conventions #105

Closed
naz opened this issue Nov 11, 2013 · 4 comments
Closed

Add .jshintrc config file to be compliant with conventions #105

naz opened this issue Nov 11, 2013 · 4 comments

Comments

@naz
Copy link

naz commented Nov 11, 2013

I think that would be a good idea to create a .jshintrc config file to describe all these conventions.

@nkbt
Copy link

nkbt commented Nov 11, 2013

Since I use same conventions, here are my JSHint configs (with comments)
https://gist.github.com/nkbt/7410960 - for NodeJS
https://gist.github.com/nkbt/7410965 - for Browser

@naz
Copy link
Author

naz commented Nov 11, 2013

It looks like those are not fully compatible with described conventions. I had a look at NodeJS config and it has "indent": 4 and here the conventions states: Use soft tabs set to 2 spaces. But they look good as a starting point.

@nkbt
Copy link

nkbt commented Nov 11, 2013

You are right, Gargol. I reckon it's the only difference though. I still prefer to have 4 spaces indent to have better visual separation.

@hshoff
Copy link
Member

hshoff commented Nov 14, 2013

Thanks for checking out the guide!

Checkout out our sublime linter config file. It uses jshint options that are compliant with the guides conventions. https://github.com/airbnb/javascript/blob/master/linters/SublimeLinter/SublimeLinter.sublime-settings

🍻

@hshoff hshoff closed this as completed Nov 14, 2013
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

No branches or pull requests

3 participants