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

Use new factory definition syntax #110

Merged
merged 2 commits into from
Mar 26, 2016
Merged

Commits on Mar 26, 2016

  1. Use new factory definition syntax

    Pros
    
    * Can use symbol view/ctags to jump to the factories more easily
    * Can add helper functions underneath the factory they belong to
    * Easier to type :)
    * If/when callbacks are introduced, this will allow you to put the
      callback functions near the factory
    
    Cons
    
    * A little more *magic* since you still call the functions with atoms,
      e.g.`build(:user)`
    paulcsmith committed Mar 26, 2016
    Configuration menu
    Copy the full SHA
    a7fc42d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    384fbec View commit details
    Browse the repository at this point in the history