Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update docstrings and docs #232

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.

Sign up for GitHub

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

Jump to bottom
Merged
alessiamarcolini merged 41 commits into from
Apr 26, 2021
Merged

Update docstrings and docs #232

alessiamarcolini merged 41 commits into from
Apr 26, 2021

Conversation

@nicolebussola
Copy link
Collaborator

@nicolebussola nicolebussola commented Feb 22, 2021

No description provided.

@coveralls
Copy link

coveralls commented Feb 22, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 1086899 on improve_docs into 2a4706d on master.

ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio requested changes Feb 23, 2021
View reviewed changes
Copy link
Member

@ernestoarbitrio ernestoarbitrio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a great improvement for our docs, but we need to run doctests at this point.

ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
ernestoarbitrio
ernestoarbitrio reviewed Feb 23, 2021
View reviewed changes
alessiamarcolini
alessiamarcolini reviewed Feb 23, 2021
View reviewed changes
alessiamarcolini
alessiamarcolini reviewed Feb 23, 2021
View reviewed changes
@alessiamarcolini

This comment has been minimized.

Sign in to view
alessiamarcolini
alessiamarcolini reviewed Feb 23, 2021
View reviewed changes
@nicolebussola nicolebussola changed the title Update docstring and docs Update docstrings and docs Feb 25, 2021
alessiamarcolini
alessiamarcolini requested changes Feb 25, 2021
View reviewed changes
@ernestoarbitrio ernestoarbitrio added the documentation Improvements or additions to documentation label Feb 26, 2021
alessiamarcolini
alessiamarcolini requested changes Feb 26, 2021
View reviewed changes
@alessiamarcolini

This comment has been minimized.

Sign in to view
@ernestoarbitrio

This comment has been minimized.

Sign in to view
alessiamarcolini
alessiamarcolini approved these changes Apr 26, 2021
View reviewed changes
Copy link
Collaborator

@alessiamarcolini alessiamarcolini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!!!!!! 🙌🎉

@alessiamarcolini alessiamarcolini merged commit b2536c8 into master Apr 26, 2021
@alessiamarcolini alessiamarcolini deleted the improve_docs branch April 26, 2021 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ernestoarbitrio ernestoarbitrio ernestoarbitrio approved these changes

@alessiamarcolini alessiamarcolini alessiamarcolini approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

0.2.5

Development

Successfully merging this pull request may close these issues.

5 participants

@nicolebussola @coveralls @alessiamarcolini @ernestoarbitrio