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

Add autosummary for the parameters module #4272

Merged
merged 2 commits into from
Jun 16, 2022

Conversation

jenshnielsen
Copy link
Collaborator

@jenshnielsen jenshnielsen commented Jun 16, 2022

  • It seems like autodocsum has an issue with inherited attributes. Worked around this by making label a property which is also cleaner anyway

@jenshnielsen jenshnielsen added the docs Related to docs improvements label Jun 16, 2022
@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #4272 (0c36f04) into master (0c36f04) will not change coverage.
The diff coverage is n/a.

❗ Current head 0c36f04 differs from pull request most recent head cc8668a. Consider uploading reports for the commit cc8668a to get more accurate results

@@           Coverage Diff           @@
##           master    #4272   +/-   ##
=======================================
  Coverage   68.35%   68.35%           
=======================================
  Files         241      241           
  Lines       30887    30887           
=======================================
  Hits        21112    21112           
  Misses       9775     9775           

@jenshnielsen jenshnielsen merged commit 15b210b into microsoft:master Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Related to docs improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants