-
Notifications
You must be signed in to change notification settings - Fork 765
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: cli auto login #4817
feat: cli auto login #4817
Conversation
3af06c0
to
db95e8c
Compare
Consider using |
Just need a one-time process server here. Would it be too heavy to start a starlette app? |
Makes sense, let's keep it |
0e826ef
to
f8d5cf4
Compare
2ffaa21
to
80ef5d7
Compare
For more information, see https://pre-commit.ci
For more information, see https://pre-commit.ci
For more information, see https://pre-commit.ci
For more information, see https://pre-commit.ci
For more information, see https://pre-commit.ci
6975510
to
d867b9e
Compare
For more information, see https://pre-commit.ci
from starlette.testclient import TestClient | ||
from typing_extensions import Annotated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doesn't seem like this change is related?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but it's just an import reorder.
What does this PR address?
support cli auto login
![image](https://private-user-images.githubusercontent.com/141706136/342576565-f560aa90-cbba-40eb-8187-05b4de8b85af.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyMDM3MzIsIm5iZiI6MTcyMDIwMzQzMiwicGF0aCI6Ii8xNDE3MDYxMzYvMzQyNTc2NTY1LWY1NjBhYTkwLWNiYmEtNDBlYi04MTg3LTA1YjRkZThiODVhZi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzA1JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwNVQxODE3MTJaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00MzY0OTk3NDdkYzBhOTcwZWI4MzMxOGVmNDM4YTU4MzUwZDQ2ODBkMDJhYjIxZjlkNDYzMjE1MzZlMDg1MTAxJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.uJM32veV7jPxJeb-L0tcqxXGk0GCAvb62xQ6PRW-Tw8)
Fixes #(issue)
Before submitting:
guide on how to create a pull request.
pre-commit run -a
script has passed (instructions)?those accordingly? Here are documentation guidelines and tips on writting docs.