-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
AirbyteLib: Improved progress print, especially in the terminal #34973
Conversation
…ive view until it is needed
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ove-progress-messaging
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.
Logic all looks good to me.
|
||
# This is a dummy secret, just to test functionality. | ||
DUMMY_SECRET = ab.get_secret("DUMMY_SECRET") |
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.
this does not seem to be used anywhere.
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.
Yes, that's correct. It's literally just there to make sure our progress printer isn't conflicting with our ability to take input from the user, which was previously an issue.
This solves some problems with the rich live view, and takes a more formal approach to (1) selecting the best redering option, and (2) creating the "live view" rendering elements only when they are needed.
This also adds friendly prints where otherwise we didn't give any feedback to the user about what was happening.
Lastly, I've updated the "run_github.py" to use a reasonably sized repo, and it runs now in <1 minute.
(Set to auto-merge on approval and passing tests.)