Skip to content

Conversation

ghandic
Copy link
Contributor

@ghandic ghandic commented Sep 8, 2021

Initial fix to switch order of preference for using docstring annotations over signature annotations in the 'docstring_sections' value of the output

…ions over signature annotations in the 'docstring_sections' value of the output
@ghandic ghandic changed the title Initial fox to switch order of preference for using docstring annotat… Initial fix to switch order of preference for using docstring annotat… Sep 8, 2021
@pawamoy
Copy link
Member

pawamoy commented Sep 10, 2021

Thanks! I just added two comments.

@ghandic
Copy link
Contributor Author

ghandic commented Sep 10, 2021

Thanks for the review, I think I have fixed the issues you've pointed out. If not then I think it might be worth adding some extra test cases of examples where it could break

@pawamoy
Copy link
Member

pawamoy commented Sep 14, 2021

Great, thanks! The comments are especially appreciated! I'll run this locally to see how it behaves, and we should be good to go.

@ghandic
Copy link
Contributor Author

ghandic commented Sep 17, 2021

Are there any fixes needed here? Or gtg?

@pawamoy
Copy link
Member

pawamoy commented Sep 21, 2021

Hey @ghandic, thank you again, I'll merge this and release it in version 0.12.0 🙂

@pawamoy pawamoy merged commit c4f6bdc into mkdocstrings:master Sep 21, 2021
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.

2 participants