Clarify Readme #12

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

jescalan commented May 2, 2012

There was a bunch of information missing in the readme and the install instructions were unclear and incomplete. This commit modifies the readme to make it much more clear and highlights issues with Spree 1.0.x and how to fix them.

-- Fixes #10

Checked it out, and I think this Pull request will make the experience way more pleasant for a newcomer to this gem.

Whitespace is a bit off right here.

Owner

jescalan replied Jun 25, 2012

do you want me to change this before you merge?

I don't have merging privileges, I was just making an observation :-)

Member

radar commented Jul 20, 2012

I will pull this in with some changes. Going to create a 1-0-stable branch off the commit you point at.

radar added a commit that referenced this pull request Jul 20, 2012

clarify readme and add a bunch of missing information
(With some changes from Ryan Bigg also)

Merges #12
Member

radar commented Jul 20, 2012

Merged with @08e148a1b27db53f6ebfd70be834b5d4a4e84c4a.

@radar radar closed this Jul 20, 2012

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