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

remove pytest warning and update setup.py #128

Merged
merged 1 commit into from
Jun 21, 2024
Merged

remove pytest warning and update setup.py #128

merged 1 commit into from
Jun 21, 2024

Conversation

vishalvvr
Copy link
Contributor

  • DeprecationWarning: datetime.datetime.utcnow() this warning creating lot of unnecessary warnings, so add get_now_str() & 'get_now()' function to /opl/date.py.
  • run flake8 and black on project
  • add development dependencies to setup.py and update README.md

@vishalvvr vishalvvr added the cleanup Cleanup stale code, docs, config, etc. label Jun 21, 2024
@vishalvvr vishalvvr requested a review from jhutar June 21, 2024 03:54
@vishalvvr vishalvvr self-assigned this Jun 21, 2024
@vishalvvr vishalvvr mentioned this pull request Jun 21, 2024
@vishalvvr vishalvvr force-pushed the test-cleanup branch 2 times, most recently from 09cb93d to d14c206 Compare June 21, 2024 04:06
- `DeprecationWarning: datetime.datetime.utcnow()` this warning creating lot of unnecessary warnings, so add `get_now_str()` &
'get_now()' function to `/opl/date.py`.
- run `flake8` and `black` on project
- add development dependencies to `setup.py` and update `README.md`
Copy link
Contributor

@jhutar jhutar left a comment

Choose a reason for hiding this comment

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

Nice!

@jhutar jhutar merged commit 81a41f1 into main Jun 21, 2024
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Cleanup stale code, docs, config, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants