search

melonjs / boilerplate

a melonJS boilerplate to create new projects (DEPRECATED)
MIT License
117 stars 88 forks source link

Whitespace #6

Closed agmcleod closed 10 years ago

agmcleod commented 10 years ago

This has kinda bugged me when working with boilerplate in the past. Most of the files used a \t, so I kept with that. Just wanted to keep it consistent.

parasyte commented 10 years ago

Not sure what @obiot wants to do here, but I am also quite frustrated with the inconsistent whitespace. My favorite setup is 4 spaces. One thing we haven't done with melonJS is defined any code style guidelines.

agmcleod commented 10 years ago

Yeah i generally use the standards of ruby, which is 2 spaces soft tabs. I've tried to keep consistent with what i've found in melonjs source, which has been 4 spaces.

obiot commented 10 years ago

Sorry I just came back home from a small trip. I'm of course fine with it and with the general idea to replaced all the inconsistent tabs in melonJS with 4 spaces (if this is what we agree on).

I guess that that's a check that could be managed by jslint ?

thanks !