Skip to content

Code refactorings#28

Closed
kashyapm94 wants to merge 5 commits intoaeron7:masterfrom
kashyapm94:code-refactoring
Closed

Code refactorings#28
kashyapm94 wants to merge 5 commits intoaeron7:masterfrom
kashyapm94:code-refactoring

Conversation

@kashyapm94
Copy link
Copy Markdown

The following changes have been made in the branch:

  • Use the original codes & write a neater version of them using python best practices
  • move the constants & URLs into their respective modules
  • Use type hints
  • bump up the version in setup.py to 1.5

What has not been done here:

  • Add polars functionality
  • Write tests (since all the functions depend on URLs, it is difficult to write 'general' tests

- move the current code to the original_rahu.py
- add the refactored code to the rahu.py
- move the urls & constants to separate modules
- bump up the version in setup.py
@aeron7
Copy link
Copy Markdown
Owner

aeron7 commented Aug 22, 2023

Do not want to merge. Many functions are community made. Don't want to tamper them.

@aeron7 aeron7 closed this Aug 22, 2023
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