-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
feat: Added logger API to Endpoint functions #507
base: main
Are you sure you want to change the base?
Conversation
Heyy @ashish-ad the check is failing due to a "linting error" in the Backend, so if you could run Also if you could update the PR template above... it's pretty self-explanatory. Finally, all the checkboxes (except the last) could be ticked if your PR manages to do the same xD |
Hi @McTechie, Thank You. |
Hey @McTechie, any updates on this PR? |
Hey @ashish-ad, I believe it would be a while for the maintainers to review the PR. Meanwhile, I'll be adding the Thank you for your valuable contribution! Kindly check back later if the PR needs any updates ✨ |
Hi @McTechie, it's been a week, we can do one thing maybe we can tag any maintainers to review the code and merge it. |
Hey @ashish-ad there wouldn't be a need for that since the notification workflow is setup on AutoDL's Slack workspace... |
Hey @McTechie, |
Hey @ashish-ad , I believe there isn't anything wrong with the PR, however, the core team has been inactive for a while. |
Pull Request
What does this PR do?
Fixes #470
What part does this affect?
Before submitting
black
and/orprettier
)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.