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

Add Shahnawaz to list of contributors. #1160

Merged
merged 2 commits into from Dec 16, 2016

Conversation

Projects
None yet
5 participants
@arokem
Member

arokem commented Dec 8, 2016

@sahmed95 : could you please check that I have your details right?

@coveralls

This comment has been minimized.

coveralls commented Dec 9, 2016

Coverage Status

Coverage remained the same at 88.004% when pulling 5a8c7d6 on arokem:add-sahmed95 into e1632e2 on nipy:master.

@codecov-io

This comment has been minimized.

codecov-io commented Dec 9, 2016

Current coverage is 85.46% (diff: 100%)

Merging #1160 into master will not change coverage

@@             master      #1160   diff @@
==========================================
  Files           214        214          
  Lines         24917      24917          
  Methods           0          0          
  Messages          0          0          
  Branches       2526       2526          
==========================================
  Hits          21296      21296          
  Misses         2989       2989          
  Partials        632        632          

Powered by Codecov. Last update e1632e2...5a8c7d6

@sahmed95

This comment has been minimized.

sahmed95 commented on 5a8c7d6 Dec 9, 2016

Thanks for the addition. The information is correct. :-)

@MarcCote

This comment has been minimized.

Contributor

MarcCote commented Dec 14, 2016

LGTM

@arokem

This comment has been minimized.

Member

arokem commented Dec 16, 2016

I added a few more small things (examples that weren't linked from the index).

@MarcCote : you have merge permissions, right? Since this is very minor, I think that you should feel comfortable to hit that green button if this looks good to you...

@coveralls

This comment has been minimized.

coveralls commented Dec 16, 2016

Coverage Status

Coverage remained the same at 88.004% when pulling f73120a on arokem:add-sahmed95 into e1632e2 on nipy:master.

@MarcCote MarcCote merged commit 70bdfe1 into nipy:master Dec 16, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 88.004%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment