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

Show progress bars #251

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

tiran
Copy link
Collaborator

@tiran tiran commented Jul 26, 2024

The bootstrap, build-sequence and download-sequence commands now use tqdm to show a progress bar. It's now easier to see how far along a long-running pipeline job has come.

@tiran tiran force-pushed the verbose-build-sequence branch from a6b1a14 to 302b861 Compare July 26, 2024 07:14
@tiran tiran changed the title Show progress bars and build-sequence duration Show progress bars Jul 26, 2024
@tiran tiran force-pushed the verbose-build-sequence branch from 302b861 to 011b241 Compare July 26, 2024 07:14
@tiran tiran marked this pull request as ready for review July 26, 2024 07:21
@tiran tiran force-pushed the verbose-build-sequence branch from 011b241 to af34399 Compare July 26, 2024 07:38
The bootstrap, build-sequence and download-sequence commands now use
`tqdm` to show a progress bar. It's now easier to see how far along a
long-running pipeline job has come.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran tiran force-pushed the verbose-build-sequence branch from af34399 to 99b96f8 Compare July 26, 2024 07:39
for entry in progress.progress(json.load(f)):
dist_name = entry["dist"]
dist_version = Version(entry["version"])
logger.info("%s: Building %s==%s", dist_name, dist_name, dist_version)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I've been trying to standardize log messages as all lower case

@mergify mergify bot merged commit c728b0c into python-wheel-build:main Jul 26, 2024
50 checks passed
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.

2 participants