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

(maint) README.md - Update "Community tag" link #492

Merged
merged 1 commit into from
Nov 26, 2015
Merged

(maint) README.md - Update "Community tag" link #492

merged 1 commit into from
Nov 26, 2015

Conversation

tirolo
Copy link
Contributor

@tirolo tirolo commented Nov 26, 2015

Changed the link from "Community tag" to "Up for Grabs"

@tirolo tirolo changed the title Tiny update for the Community tag link README file: Update for the Community tag link Nov 26, 2015
@ferventcoder
Copy link
Member

Two of the tests have issues because C:\Windows versus C:\windows - not something you need to fix though.

@ferventcoder
Copy link
Member

However, your commit is not quite right. It should either contain a GH number or (maint). Plus a commit message body wouldn't hurt (but is not necessary).

@tirolo tirolo changed the title README file: Update for the Community tag link (maint) README.md - Update "Community tag" link Nov 26, 2015
@tirolo
Copy link
Contributor Author

tirolo commented Nov 26, 2015

Looks better now? :)
(ops, just realised by looking at others commit that the (maint) should go on the title not in the body :P )

Changed the link from "Community tag" to "Up for Grabs"
@gep13
Copy link
Member

gep13 commented Nov 26, 2015

@tirolo looks good to me! 👍

ferventcoder added a commit that referenced this pull request Nov 26, 2015
(maint) README.md - Update "Community tag" link
@ferventcoder ferventcoder merged commit 1db4f07 into chocolatey:master Nov 26, 2015
@ferventcoder
Copy link
Member

Thanks!

@tirolo tirolo deleted the patch-1 branch November 26, 2015 14:39
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.

3 participants