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

Update API Documentation: Change Parameter Type from "string" to "str" in Comments for Python Consistency #460

Merged
merged 3 commits into from
Nov 26, 2024

Conversation

wDushanin
Copy link
Contributor

Provide a summary of the proposed changes, describe tests and documentation, and review the acknowledgement below.

Summary

This pull request addresses an update to the API documentation in the fragility_curve.py file. Specifically, the data type annotation for the name parameter was updated from "string" to "str" to provide a direct link to Python documentation, improving usability and clarity.

Tests and documentation

This change impacts only the documentation and does not modify the functionality of the code. Therefore, no additional tests are required.

Acknowledgement

By contributing to this software project, I acknowledge that I have reviewed the software quality assurance guidelines and that my contributions are submitted under the Revised BSD License.

In the class "State": 
In the comments, change "string" to "str" so that the parameter type is linked to the python docs in the API documentation.
@coveralls
Copy link

Coverage Status

coverage: 84.364% (+0.02%) from 84.341%
when pulling 758451f on wDushanin:main
into eda4c89 on USEPA:main.

@kaklise kaklise merged commit 0badb0b into USEPA:main Nov 26, 2024
41 checks passed
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.

3 participants