Auto-DL
Auto-DL copied to clipboard
fix: update backend app logging
Pull Request
What does this PR do?
Fixes #471
Added logs in backend app which replace print statements.
What part does this affect?
- [ ] FrontEnd.
- [x] BackEnd.
- [ ] Documentation.
- [ ] Other. (Please specify below)
Before submitting
- [x] Was this discussed/approved via a GitHub issue or slack?
- [x] Did you read the contributor guideline?
- [x] Did you ensure that there aren't any other open Pull Requests for the same update/change?
- [x] Did you make sure the title is self-explanatory and the description concisely explains the PR?
- [x] Did you make sure your PR does only one thing, instead of bundling different changes together?
- [ ] Did you make sure the code is clean and docstrings have been added or updated as required?
- [ ] Did you make sure the code is linted/formatted locally prior to submission? (using
black
and/orprettier
) - [ ] Did you make sure to update the documentation with your changes? (if necessary)
PR review
Anyone in the community is free to review the PR once the tests have passed.
Thank you for contributing to AutoDL. We look forward to your continued support.
Hi @ShreyaPrasad1209,
Could you please run black .
to autolint your code?
Also there's a conflict in utils.
Please look into it as well :)
Hey, @ShreyaPrasad1209 have you linted the backend as suggested by @ADI10HERO? :eyes:
Additionally, there appears to be a change in the docs
in addition to updating the logs
, could we try to handle both different issues in separate PRs?
There also appears to be a conflict so let's resolve them before we're good to merge? xD
Closing due to inactivity