-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Added An Upgrade Guide And Better Usage Docs #193
Conversation
@scottrobertson I'd say I'm pretty much done writing it now. You can see the full file in all it's glory here: https://github.com/GrahamCampbell/factory-muffin/blob/upgrading/UPGRADING.md. What do you think? |
I am on my phone out of the house so I can't really get into it fully. It looks awesome though. There are back few cases where you are using Factory directly instead of the Facade. |
@scottrobertson Fixed, and I've added more usage docs to the readme too. You can see the readme here: https://github.com/GrahamCampbell/factory-muffin/tree/upgrading. |
@scottrobertson Are you able to take a look at this today? |
This all looks awesome :) it all good to merge? |
Added An Upgrade Guide And Better Usage Docs
In response to #67 and #186.