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

Enhance Docstrings in aepsych/factory #428

Closed

Conversation

yalsaffar
Copy link
Contributor

One of several PRs addressing issue #366 to improve docstring coverage.

Improves documentation in aepsych/factory for better clarity and consistency.

  • Adds missing docstrings to functions and methods across factory modules.
  • Updates existing docstrings with refined type hints and a unified structure.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 31, 2024
Copy link
Contributor

@JasonKChow JasonKChow left a comment

Choose a reason for hiding this comment

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

Need rebase then good.

@facebook-github-bot
Copy link
Contributor

@JasonKChow has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@JasonKChow merged this pull request in 10100e4.

JasonKChow pushed a commit that referenced this pull request Nov 25, 2024
Summary:
One of several PRs addressing issue #366 to improve docstring coverage.

Improves documentation in `aepsych/factory` for better clarity and consistency.
- Adds missing docstrings to functions and methods across factory modules.
- Updates existing docstrings with refined type hints and a unified structure.

Pull Request resolved: #428

Reviewed By: crasanders

Differential Revision: D65950795

Pulled By: JasonKChow

fbshipit-source-id: 0a18cf0ca76d05574718f11352b8b0587d54f912
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants