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

docs: NamedContract and MoccasinAccount class ref for #86 #208

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

s3bc40
Copy link
Contributor

@s3bc40 s3bc40 commented Feb 20, 2025

Related Issue: #86


  • Docstring added for NamedContract and MoccasinAccount
  • Refactor docstring param and ivar for clarity
  • Moved docstring params from __init__ to class docstring to show in official doc

I adapted param and ivar because I understood that @dataclass make the __init__ automatically from the variables declared in the class. So they are not variables used during runtime but actual params to init the class.

- Docstring added for ``NamedContract`` and ``MoccasinAccount``
- Refactor docstring ``param`` and ``ivar`` for clarity
- Moved docstring params from ``__init__`` to class docstring to show in official doc
@s3bc40 s3bc40 changed the title DOCS - NamedContract and MoccasinAccount class ref DOCS - NamedContract and MoccasinAccount class ref for #86 Feb 20, 2025
@s3bc40 s3bc40 changed the title DOCS - NamedContract and MoccasinAccount class ref for #86 docs: NamedContract and MoccasinAccount class ref for #86 Feb 20, 2025
@s3bc40 s3bc40 mentioned this pull request Feb 20, 2025
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.

1 participant