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

Adding "campuses" as plural of "campus" #364

Merged
merged 2 commits into from
Dec 21, 2014
Merged

Adding "campuses" as plural of "campus" #364

merged 2 commits into from
Dec 21, 2014

Conversation

jermdavis
Copy link

The word "campus" was not being pluralised correctly - was just returning "campus". So I have added rules for plural/singular and an entry to the appropriate test data.

@MehdiK
Copy link
Member

MehdiK commented Dec 20, 2014

Thanks. Please add the PR to release notes.

@jermdavis
Copy link
Author

Sorry about that - didn't realise you needed me to update that document. I've added PR364 - hopefully that's the format you want?

@MehdiK
Copy link
Member

MehdiK commented Dec 21, 2014

No problems @jermdavis! Thanks for the addition.

MehdiK added a commit that referenced this pull request Dec 21, 2014
Adding "campuses" as plural of "campus"
@MehdiK MehdiK merged commit d5b6587 into Humanizr:master Dec 21, 2014
@MehdiK
Copy link
Member

MehdiK commented Dec 21, 2014

You might refer to the contrib guideline for your future contributions. Thanks.

@MehdiK
Copy link
Member

MehdiK commented Jan 26, 2015

Thanks for the contribution. This is now released to NuGet.

@jermdavis
Copy link
Author

Cheers - Thank you for maintaining such a useful package.

Jeremy Davis
DJDavis@iee.org

On 26 January 2015 at 00:25, Mehdi Khalili notifications@github.com wrote:

Thanks for the contribution. This is now released to NuGet.


Reply to this email directly or view it on GitHub
#364 (comment).

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.

None yet

2 participants