-
Notifications
You must be signed in to change notification settings - Fork 244
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
Style Guides. #69
Comments
This sounds like a great idea. I think using someone else's work will be best to save us time, and we can always modify as needed. @simonv3 can you throw your guides of choice onto the wiki? |
We must enforce a good style guide. For Ruby, the defacto standard is this one: https://github.com/bbatsov/ruby-style-guide I might throw Hound-CI up in a few days to automate the enforcement process. |
Great, I'll put those up when I get a chance. Rick, thanks for the link -
|
Added style guides to the wiki. I didn't copy them over completely because I figured we didn't need to do that until we started needing to. |
Awesome. Better than my wild west of style 😆. We should practice checking for these things in PRs. |
It makes sense to create a series of code style guides for HTML/JS/CSS.
We can make our own, or just steal some from others.
CSS options: http://css-tricks.com/css-style-guides/
JS the Airbnb one is pretty good: https://github.com/airbnb/javascript
These could just be pages on the wiki for people to refer to, and could evolve as the codeset progresses.
It would also be a good place to define styles on top of the Foundation presets.
Thoughts?
The text was updated successfully, but these errors were encountered: