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

Replaced DiPy with DIPY in readme.rst and info.py #524

Merged
merged 2 commits into from Dec 28, 2014

Conversation

Projects
None yet
3 participants
@Garyfallidis
Member

Garyfallidis commented Dec 28, 2014

No description provided.

@samuelstjean

This comment has been minimized.

Contributor

samuelstjean commented Dec 28, 2014

While you are at it, the last line of the readme starts with a small letter, and the NIPY dipy stuff in the middle is ugly also, I suggest either all small or all capitals.

@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented Dec 28, 2014

Thx, corrected.

@samuelstjean

This comment has been minimized.

Contributor

samuelstjean commented Dec 28, 2014

Also, you should add that the shore code is under gpl and a disclaimer like in the readme of the function itself, since the last line only says bsd licensed code.

@arokem

This comment has been minimized.

Member

arokem commented Dec 28, 2014

That's already done, here: #499

On Sun, Dec 28, 2014 at 7:28 AM, Samuel St-Jean notifications@github.com
wrote:

Also, you should add that the shore code is under gpl and a disclaimer
like in the readme of the function itself, since the last line only says
bsd licensed code.


Reply to this email directly or view it on GitHub
#524 (comment).

arokem added a commit that referenced this pull request Dec 28, 2014

Merge pull request #524 from Garyfallidis/release-0.8.checks
Replaced DiPy with DIPY in readme.rst and info.py

@arokem arokem merged commit 3b67d88 into nipy:master Dec 28, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@samuelstjean

This comment has been minimized.

Contributor

samuelstjean commented Dec 28, 2014

I meant explicitely to the readme, as done in the shore notice. It's a bit buried under there that people might not notice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment