Skip to content

fix drs-pull progress bar display behavior - #312

Merged
mfshao merged 3 commits into
masterfrom
fix/drs-pull-progress-bar
Aug 18, 2026
Merged

mfshao merged 3 commits into
masterfrom
fix/drs-pull-progress-bar

Conversation

@mfshao

@mfshao mfshao commented Aug 18, 2026 •

Copy link
Copy Markdown
Contributor

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

  • the --no-progress arg now works as expected for drs-pull object and drs-pull objects commands

Improvements

  • by default drs-pull object and drs-pull objects commands now shows progress bar when downloading, unless you set the --no-progress arg to suppress it

Dependency updates

Deployment changes

@github-actions

Copy link
Copy Markdown

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_dbgap.py 4 0 1 5
tests/test_ras_passport.py 0 0 2 2
tests/test_data_upload.py 8 0 1 9
tests/test_graph_submit_and_query.py 12 1 1 14
tests/test_presigned_url.py 7 0 0 7
tests/test_centralized_auth.py 5 0 0 5
tests/test_google_data_access.py 1 0 0 1
tests/test_gen3_sdk.py 1 0 0 1
tests/test_audit_service.py 1 0 0 1
tests/test_drs_endpoint.py 2 0 0 2
TOTAL 41 1 5 47

Test summary after rerunning failed integration tests

filepath passed SUBTOTAL
tests/test_graph_submit_and_query.py 1 1
TOTAL 1 1

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@mfshao
mfshao merged commit 8e0e5ea into master Aug 18, 2026
19 checks passed
@mfshao
mfshao deleted the fix/drs-pull-progress-bar branch August 18, 2026 18:15
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