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

add logging overriding #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

07pepa
Copy link

@07pepa 07pepa commented Jun 18, 2024

sometimes its nice to have your custom logging in fastapi so i added passing it to uvicorn and fallbacking to uvicorn default one if nothing is passed in

@07pepa 07pepa marked this pull request as draft June 18, 2024 16:11
@07pepa 07pepa marked this pull request as ready for review June 19, 2024 07:48
@Leon0824
Copy link

+1

2 similar comments
@FelixBehne
Copy link

+1

@wchaws
Copy link

wchaws commented Jul 29, 2024

+1

@saligrama
Copy link

+1

@07pepa
Copy link
Author

07pepa commented Aug 19, 2024

@wchaws can you please add tag feature to fix failing test? thanks

@wchaws
Copy link

wchaws commented Aug 20, 2024

@07pepa I'm not the maintainer for this project. I just accidentally approved this PR.

@07pepa
Copy link
Author

07pepa commented Aug 20, 2024

ok i thought you would have some extra rights if you could approve

patrick91 added a commit to patrick91/fastapi-cli that referenced this pull request Sep 23, 2024
…_existing_app

✨ Allow to use an existing app
@dineshnauti
Copy link

+1

@jrs53
Copy link

jrs53 commented Oct 22, 2024

Is this PR just missing label "feature"? @07pepa @wchaws Are you able to add it?

Thanks

@07pepa
Copy link
Author

07pepa commented Oct 22, 2024

Is this PR just missing label "feature"? @07pepa @wchaws Are you able to add it?

Thanks

Yes i am unable as author of PR to change/add label. Label is only thing that is missing.

i tried even modifying labeler and no luck #110

we probably need to ask @tiangolo how to do this (or to add label)

@vitaliy-sk
Copy link

+1

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.

8 participants