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

Update docs (closes #138) #139

Merged
merged 3 commits into from
Mar 12, 2021
Merged

Update docs (closes #138) #139

merged 3 commits into from
Mar 12, 2021

Conversation

dmey
Copy link
Contributor

@dmey dmey commented Mar 11, 2021

This adds the comments as suggested in #138.

@dmey dmey marked this pull request as ready for review March 11, 2021 17:34
@dmey
Copy link
Contributor Author

dmey commented Mar 11, 2021

@wimvanderbauwhede I have updated the docs to include your comments-- please let me know if you are happy with them. @samoliverowens @tomgrylls please see if this works with you too or modify accordingly.

@dmey dmey changed the title Update docs Update docs (closes #138) Mar 11, 2021
@dmey dmey merged commit 4d048be into master Mar 12, 2021
@dmey dmey deleted the dmey/docs branch March 12, 2021 16:48
@wimvanderbauwhede
Copy link

@dmey Yes, I am happy with these changes, thanks!

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.

3 participants