Skip to content

correct uses of :param name: and :return: in docstrings to generate API docs in MarkDown format#4129

Merged
ocaisa merged 3 commits intoeasybuilders:developfrom
branfosj:fix_param
Nov 27, 2022
Merged

correct uses of :param name: and :return: in docstrings to generate API docs in MarkDown format#4129
ocaisa merged 3 commits intoeasybuilders:developfrom
branfosj:fix_param

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Nov 27, 2022

One of the markdown API document generators I am trying:

  • blows up on :param:
  • does not recognise :params ...
  • blows up on :param name description (missing : after the parameter name)
  • blows up on :return ...

@branfosj branfosj changed the title use :param name: consistently in docstrings correct uses of :param name: and :return: in docstrings Nov 27, 2022
ocaisa
ocaisa previously approved these changes Nov 27, 2022
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM

@ocaisa ocaisa enabled auto-merge November 27, 2022 10:10
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

Still LGTM

@ocaisa ocaisa merged commit 872df2b into easybuilders:develop Nov 27, 2022
@branfosj branfosj deleted the fix_param branch November 27, 2022 11:19
@boegel boegel added the bug fix label Dec 18, 2022
@boegel boegel changed the title correct uses of :param name: and :return: in docstrings correct uses of :param name: and :return: in docstrings to generate API docs in MarkDown format Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants