Skip to content

disable progressbar for mass pdsc downloads, issue #250 #521

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.

Sign up for GitHub

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

Jump to bottom
Merged
jkrech merged 1 commit into from
May 7, 2025
Merged

disable progressbar for mass pdsc downloads, issue #250 #521

jkrech merged 1 commit into from
May 7, 2025

Conversation

@bgn42
Copy link
Collaborator

@bgn42 bgn42 commented May 7, 2025 •
edited by jkrech
Loading

Fixes

Changes

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@qlty-cloud-legacy
Copy link

qlty-cloud-legacy bot commented May 7, 2025

Code Climate has analyzed commit 78e241e and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 90.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 60.1% (0.0% change).

View more on Code Climate.

@bgn42 bgn42 requested a review from jkrech May 7, 2025 11:16
jkrech
jkrech approved these changes May 7, 2025
View reviewed changes
Copy link
Member

@jkrech jkrech left a comment

Choose a reason for hiding this comment

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

LGTM

@jkrech jkrech merged commit 154c4e9 into main May 7, 2025
20 checks passed
@jkrech jkrech deleted the noProgressBar branch May 7, 2025 11:57
@jkrech jkrech linked an issue May 7, 2025 that may be closed by this pull request
[cpackget] Confusing unaligned output cpackget update-index -a due to concurrent downloads #250
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jkrech jkrech jkrech approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[cpackget] Confusing unaligned output cpackget update-index -a due to concurrent downloads

3 participants

@bgn42 @jkrech