Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Update doc #58

Closed
pypingou opened this issue Mar 5, 2015 · 5 comments · Fixed by #65
Closed

Update doc #58

pypingou opened this issue Mar 5, 2015 · 5 comments · Fixed by #65
Labels
easyfix

Comments

@pypingou
Copy link
Member

pypingou commented Mar 5, 2015

I was pointed out that the doc isn't up to date, they do not work if you follow them verbatim, we should improve on this :)
#57

Ralph fixed one thing on #59 (see below) but there might be more, we should go through and fix them.
@ralphbean
Copy link
Contributor

Part of this is taken care of by #59.

@zeus9
Copy link

zeus9 commented Mar 7, 2015

I would like to contribute

@vatsala vatsala mentioned this issue Mar 28, 2015
Merged
@vatsala
Copy link
Contributor

vatsala commented Mar 28, 2015

PR #65

@pypingou
Copy link
Member Author

The goal of this ticket was not to change the doc but to adjust it to see what is missing and what needs adjustments.
Thank you for your work on improving the current wording, there were apparently a number of mistakes, but PR#65 does not solve this ticket :)

@vatsala
Copy link
Contributor

vatsala commented Mar 28, 2015

@pypingou Ok, I will run through the doc again and add whatever is missing and whatever needs adjustments.

Also replying to your various code review comments on the PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
easyfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Some improvements to the documentation vatsala/nuancier
4 participants
@vatsala @ralphbean @pypingou @zeus9