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

[MRG+1] Upgrade CoC and mention it after main contributing docs #2469

Merged
merged 3 commits into from Jan 8, 2017

Conversation

eliasdorneles
Copy link
Member

Attempt to address latest comments on #1645

The idea is we want to show people how to contribute before showing them the Code of Conduct (which should be obvious for most people).

@codecov-io
Copy link

codecov-io commented Dec 27, 2016

Current coverage is 83.44% (diff: 100%)

Merging #2469 into master will not change coverage

@@             master      #2469   diff @@
==========================================
  Files           161        161          
  Lines          8779       8779          
  Methods           0          0          
  Messages          0          0          
  Branches       1288       1288          
==========================================
  Hits           7326       7326          
  Misses         1205       1205          
  Partials        248        248          

Powered by Codecov. Last update cc06b6b...b7dd089

@eliasdorneles eliasdorneles changed the title Mention contributing document before CoC in the README Upgrade CoC and mention it after main contributing docs Dec 27, 2016
@redapple
Copy link
Contributor

redapple commented Jan 3, 2017

Looks good to me. Thanks @eliasdorneles !

@redapple redapple changed the title Upgrade CoC and mention it after main contributing docs [MRG+1] Upgrade CoC and mention it after main contributing docs Jan 3, 2017
@kmike kmike merged commit e7d0dae into master Jan 8, 2017
@redapple redapple deleted the update-readme branch January 10, 2017 09:43
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

4 participants