Skip to content

Conversation

@Leemoonsoo
Copy link
Member

What is this PR for?

Adding code of conduct link http://www.apache.org/foundation/policies/conduct.html to our contribution guide, as well as http://www.apache.org/foundation/how-it-works.html

What type of PR is it?

Improvement

Todos

  • - Add link to Code of conduct
  • - Add link to the Apache way.

Is there a relevant Jira issue?

no

How should this be tested?

N/A

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@Leemoonsoo Leemoonsoo changed the title Add link to code of conduct, from CONTRIBUTING.md Add link to code of conduct / how it works, from CONTRIBUTING.md Feb 10, 2016
@jongyoul
Copy link
Member

LGTM

CONTRIBUTING.md Outdated

Before you start, please read the [Code of Conduct](http://www.apache.org/foundation/policies/conduct.html) carefully, familiarise yourself with it and refer to it whenever you need it.

For those of who are not familiar with Apache project, understanding how [How it works](http://www.apache.org/foundation/how-it-works.html) would be quite helpful.
Copy link
Contributor

Choose a reason for hiding this comment

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

@Leemoonsoo How about deleting "how" ?( i mean the former one )
Except this, LGTM!

Copy link
Contributor

Choose a reason for hiding this comment

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

For those of who are not -> For those of you who are not

And like it was said, there is a double how in that sentence

@Leemoonsoo
Copy link
Member Author

@jongyoul @AhyoungRyu @corneadoug Thanks for the review. I have addressed the comment.

@Leemoonsoo
Copy link
Member Author

CI failing is irrelevant. Merge if there're no more discussions.

@asfgit asfgit closed this in b391317 Feb 15, 2016
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.

4 participants