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

update docs to include collection humanization #271

Merged
merged 4 commits into from
May 21, 2014

Conversation

justin-edwards
Copy link
Contributor

No description provided.

@@ -14,6 +14,7 @@ Humanizer meets all your .NET needs for manipulating and displaying strings, enu
- [Dehumanize Enums](#dehumanize-enums)
- [Humanize DateTime](#humanize-datetime)
- [Humanize TimeSpan](#humanize-timespan)
- [Humainze Collections](#humanize-collections)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo on the link title.

new SomeClass { SomeInt = 3, SomeString = "Three" }
};

collection.Humanize() // "Specific String, Specific String, and Specific String"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add some spaces before the outputs so the outputs line up nicely?

@justin-edwards
Copy link
Contributor Author

See what happens when you have programmers write words instead of code?! 😛

@MehdiK
Copy link
Member

MehdiK commented May 21, 2014

Haha :)

There is still a Humainze typo in your description and I think the last sentence might read better as "The default separator for English is "and" but you could pass a different separator into Humanize too."

@justin-edwards
Copy link
Contributor Author

Oh man I am literally the worst. Fixed.

MehdiK added a commit that referenced this pull request May 21, 2014
update docs to include collection humanization
@MehdiK MehdiK merged commit 1d5710c into Humanizr:master May 21, 2014
@justin-edwards justin-edwards deleted the humanize-collections-readme branch May 21, 2014 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants