Skip to content

DRS changes for _list_object() and DownloadManager.download()#304

Merged
paulineribeyre merged 23 commits into
masterfrom
fix/drs_issues
Jun 10, 2026
Merged

DRS changes for _list_object() and DownloadManager.download()#304
paulineribeyre merged 23 commits into
masterfrom
fix/drs_issues

Conversation

@krishnaa05

Copy link
Copy Markdown
Contributor

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

Improvements

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.

Comment thread gen3/tools/download/drs_download.py Outdated
@github-actions

Copy link
Copy Markdown

Integration Tests

filepath passed skipped SUBTOTAL
tests/test_presigned_url.py 7 0 7
tests/test_graph_submit_and_query.py 13 1 14
tests/test_data_upload.py 8 1 9
TOTAL 28 2 30

Please find the detailed integration test report here

Please find the Github Action logs here

Comment thread gen3/tools/download/drs_download.py Outdated
paulineribeyre
paulineribeyre previously approved these changes Jun 10, 2026

@paulineribeyre paulineribeyre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

other than that looks good

paulineribeyre
paulineribeyre previously approved these changes Jun 10, 2026
paulineribeyre
paulineribeyre previously approved these changes Jun 10, 2026
@github-actions

Copy link
Copy Markdown

Integration Tests

filepath passed skipped SUBTOTAL
tests/test_presigned_url.py 7 0 7
tests/test_graph_submit_and_query.py 13 1 14
tests/test_data_upload.py 8 1 9
TOTAL 28 2 30

Please find the detailed integration test report here

Please find the Github Action logs here

@paulineribeyre paulineribeyre merged commit 1dfcc76 into master Jun 10, 2026
17 checks passed
@paulineribeyre paulineribeyre deleted the fix/drs_issues branch June 10, 2026 18:01
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