Skip to content

Task/docstrings#13

Open
simonamaggio wants to merge 9 commits intomainfrom
task/docstrings
Open

Task/docstrings#13
simonamaggio wants to merge 9 commits intomainfrom
task/docstrings

Conversation

@simonamaggio
Copy link
Copy Markdown
Contributor

@simonamaggio simonamaggio commented Apr 14, 2021 •
edited
Loading

  • Updated/Fixed/Added docstrings
  • rm non existing input argument in doc examples

Click on the different files listed in this Updated Documentation to see the rendering of the docstrings.

@simonamaggio simonamaggio added documentation Improvements or additions to documentation and removed WIP labels Apr 14, 2021
@simonamaggio simonamaggio requested review from AgatheG and du-phan April 14, 2021 20:05
@AgatheG
Copy link
Copy Markdown
Contributor

AgatheG commented Apr 18, 2021

I'd say d8ae9ba (and to a lesser extent, 636fe17) are more urgent commits/unrelated to the PR at hand. They could be pushed to main directly 🙂

@AgatheG AgatheG removed their request for review January 6, 2023 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@du-phan du-phan Awaiting requested review from du-phan

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@simonamaggio @AgatheG