Skip to content

Instantly share code, notes, and snippets.

@SimplGy

SimplGy/John Papa Style Guide Review.md

Last active October 9, 2015 21:01
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save SimplGy/24ca903d7c5c56a3d215 to your computer and use it in GitHub Desktop.
Save SimplGy/24ca903d7c5c56a3d215 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
John Papa Style Guide Review.md
  • Single Responsibility -- one thing per file
  • IIFE -- free from coffeescript
  • Modules -- declare once, nest namespaces
  • Controllers -- name your fns, use controllerAs, vm = this, public methods at top of file
  • Services -- delegate all possible behavior to here for easier testing
  • Factories -- api up top, use promises for data calls
  • Data Services
  • Directives
  • Resolving Promises for a Controller
  • Manual Annotating for Dependency Injection
  • Minification and Annotation
  • Exception Handling
  • Naming
  • Application Structure LIFT Principle
  • Application Structure
  • Modularity
  • Startup Logic
  • Angular $ Wrapper Services
  • Testing
  • Animations
  • Comments
  • JSHint
  • JSCS
  • Constants
  • File Templates and Snippets
  • Yeoman Generator
  • Routing
  • Task Automation
  • Filters
  • Angular Docs
  • Contributing
  • License
@SimplGy
Copy link
Author

SimplGy commented Oct 9, 2015

We should look at whether naming the functions passed in when defining directives/controllers/services actually helps the debug/stack trace. CS doesn't have func declarations, and I'm not sure if named function variables have the same benefit.

@SimplGy
Copy link
Author

SimplGy commented Oct 9, 2015

I like lift, though it's made to be likeable. Linking rules back to intent is good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment