Skip to content
This repository has been archived by the owner. It is now read-only.

Update README to include details of apps already using gem #25

Merged
merged 2 commits into from Jun 18, 2014
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jun 18, 2014

  • To help users/apps switching to the gem, it’s good to know where
    there are good examples
  • Added after a conversation with @alicebartlett
* To help users/apps switching to the gem, it’s good to know where
there are good examples
@alext
Copy link
Contributor

@alext alext commented Jun 18, 2014

A concern about this is that it will be very quickly out of date. I wonder whether it's better to just link to apps already using it, and not mention the apps that will use it in future.

@fofr
Copy link
Contributor Author

@fofr fofr commented Jun 18, 2014

If that happens quickly I'll be happy. Also, I like publicly committing to things.

@bradwright
Copy link
Contributor

@bradwright bradwright commented Jun 18, 2014

I think @alext is right here. Examples of current use is fine.

* This list would otherwise become out of date
@fofr
Copy link
Contributor Author

@fofr fofr commented Jun 18, 2014

Updated.

alext added a commit that referenced this pull request Jun 18, 2014
Update README to include details of apps already using gem
@alext alext merged commit 5a77287 into master Jun 18, 2014
1 check passed
1 check passed
default "Build #34 succeeded on Jenkins"
Details
@alext alext deleted the apps-using-gem branch Jun 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.