Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

DOC: Add __init__ documentation to class documentation #263

Merged
merged 1 commit into from
Dec 8, 2024

Conversation

jhlegarreta
Copy link
Collaborator

Add __init__ documentation to class documentation:

  • Add a function to the documentation configuration file that processes the signatures and adds the __init__ method signature to the class signatures.
  • Make the class documentation contain the documentation from both the class and the __ini__ method setting the autoclass_content option to both.

Remove the short description of the __init__ methods so that the class documentation reads seamlessly.

Add `__init__` documentation to class documentation:
- Add a function to the documentation configuration file that processes
  the signatures and adds the `__init__` method signature to the class
  signatures.
- Make the class documentation contain the documentation from both the
  class and the `__ini__` method setting the `autoclass_content` option
  to `both`.

Remove the short description of the `__init__` methods so that the class
documentation reads seamlessly.
@jhlegarreta jhlegarreta force-pushed the DocumentClassWithInit branch from 29c1eb9 to 5d10d90 Compare December 8, 2024 02:22
@jhlegarreta
Copy link
Collaborator Author

Local check:

eddymotion_class_init_signature_doc

Copy link

codecov bot commented Dec 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.24%. Comparing base (f10fc12) to head (5d10d90).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #263   +/-   ##
=======================================
  Coverage   66.24%   66.24%           
=======================================
  Files          19       19           
  Lines         942      942           
  Branches      119      119           
=======================================
  Hits          624      624           
  Misses        274      274           
  Partials       44       44           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@effigies effigies left a comment

Choose a reason for hiding this comment

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

Looks nice!

@effigies effigies merged commit f78bb19 into nipreps:main Dec 8, 2024
8 checks passed
@jhlegarreta jhlegarreta deleted the DocumentClassWithInit branch December 8, 2024 20:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants