Skip to content

Add api doc#88

Merged
SarahAlidoost merged 5 commits intomainfrom
add_api_doc
Jun 10, 2025
Merged

Add api doc#88
SarahAlidoost merged 5 commits intomainfrom
add_api_doc

Conversation

@SarahAlidoost
Copy link
Member

closes #65

@SarahAlidoost SarahAlidoost marked this pull request as ready for review June 5, 2025 11:07
@SarahAlidoost SarahAlidoost requested a review from rogerkuou June 5, 2025 11:07
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 5, 2025

@SarahAlidoost
Copy link
Member Author

@rogerkuou Some of the changes in this PR are related to fixing the style of docstring google vs numpy. So, here I added a checker to toml. However, it seems that we dont have an action for running ruff.

Copy link
Member

@rogerkuou rogerkuou left a comment

Choose a reason for hiding this comment

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

Thanks Sarah! This looks all good! Please merge

I made #89 as a reminder for your comment of adding ruff GH workflow .

@SarahAlidoost SarahAlidoost merged commit 93b4d47 into main Jun 10, 2025
15 checks passed
@SarahAlidoost SarahAlidoost deleted the add_api_doc branch June 10, 2025 08:14
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.

Add API documentation

2 participants