Auto-DL icon indicating copy to clipboard operation
Auto-DL copied to clipboard

fix: update backend app logging

Open ShreyaPrasad1209 opened this issue 2 years ago • 1 comments

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/or prettier)
  • [ ] 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.

ShreyaPrasad1209 avatar Mar 17 '22 09:03 ShreyaPrasad1209

Hi @ShreyaPrasad1209, Could you please run black . to autolint your code? Also there's a conflict in utils.

Please look into it as well :)

ADI10HERO avatar Mar 17 '22 09:03 ADI10HERO

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

McTechie avatar Oct 09 '22 05:10 McTechie

Closing due to inactivity

McTechie avatar Oct 15 '22 03:10 McTechie