Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

docs(contributing): add link to commit conventions #4161

Closed
wants to merge 1 commit into from

Conversation

wesleycho
Copy link
Contributor

  • Add commit convention link to document

This resolves #3602.

@wesleycho wesleycho added this to the 0.13.4 (Performance) milestone Aug 10, 2015
@@ -9,7 +9,7 @@ StackOverflow is a much better place to ask questions since:
* questions and answers stay available for public viewing so your question / answer might help someone else
* SO voting system assures that the best answers are prominently visible.

To save your and our time we will be systematically closing all the issues that are request for general support and redirecting people to StackOverflow.
To save your and our time we will be systematically closing all the issues that are request for general support and redirecting people to StackOverflow.
Copy link
Contributor

Choose a reason for hiding this comment

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

What do you think about changing To save your and our time to To save everyone's time?

Please change request to requests.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't change the text at all - I think my editor just chopped off useless whitespace.

I think the former point, although a little verbose, is a little less harsh than the proposed amendment.

I'll change the request part though.

- Add commit convention link to document
@@ -9,7 +9,7 @@ StackOverflow is a much better place to ask questions since:
* questions and answers stay available for public viewing so your question / answer might help someone else
* SO voting system assures that the best answers are prominently visible.

To save your and our time we will be systematically closing all the issues that are request for general support and redirecting people to StackOverflow.
To save your and our time we will be systematically closing all the issues that are requests for general support and redirecting people to StackOverflow.
Copy link
Contributor

Choose a reason for hiding this comment

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

Yea, I was going back and forth on that and was wondering if it would come across as harsh. You just never know what mood someone is going to be in at any given time.

@icfantv
Copy link
Contributor

icfantv commented Aug 10, 2015

👍

@wesleycho wesleycho closed this in 3c06832 Aug 10, 2015
@wesleycho wesleycho deleted the docs/contributing branch August 10, 2015 04:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AngularJS Git Commit Message Conventions
2 participants