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

refactored codebase a/c napoleon style #979

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

RahulSankhla312
Copy link
Contributor

Changed the whole codebase Refactored codebase to consistently use Napoleon-style docstrings.
Improves code readability and maintainability by providing clear function and class descriptions.

@RahulSankhla312
Copy link
Contributor Author

@mr-tz @williballenthin I have made changes using the isort , for the code style.

@mr-tz
Copy link
Collaborator

mr-tz commented Mar 19, 2024

please fix isort/black issues first

@RahulSankhla312
Copy link
Contributor Author

Corrected the isort and black issues @mr-tz

@mr-tz
Copy link
Collaborator

mr-tz commented Mar 19, 2024

code style looks good now, this will take a bit to go through

also other tests fail, need to look into those

@mr-tz
Copy link
Collaborator

mr-tz commented Mar 20, 2024

Can you look into the test failures, please?

@RahulSankhla312
Copy link
Contributor Author

@mr-tz Sure ill look into it.

@mr-tz
Copy link
Collaborator

mr-tz commented Apr 26, 2024

Can you rebase this to master to (maybe) fix the test failures, please? Otherwise, we have to look closer at them.

@RahulSankhla312
Copy link
Contributor Author

@mr-tz Sure i will take a look.

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.

None yet

2 participants