Skip to content

Fix wrap.py logs#298

Merged
paulineribeyre merged 2 commits intomasterfrom
fix-logs
Apr 20, 2026
Merged

Fix wrap.py logs#298
paulineribeyre merged 2 commits intomasterfrom
fix-logs

Conversation

@paulineribeyre
Copy link
Copy Markdown
Contributor

Fix for this bug:

TypeError: not all arguments converted during string formatting
Call stack:
  [...]
  File ".../site-packages/gen3/tools/wrap.py", line 35, in run_command
    logger.error(f"ERROR while running '{cmd}':", e)
Message: "ERROR while running '[...]':"
Arguments: (CalledProcessError(1, [...]),)

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

@paulineribeyre paulineribeyre requested a review from nss10 April 20, 2026 20:56
@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.

nss10
nss10 previously approved these changes Apr 20, 2026
@github-actions
Copy link
Copy Markdown

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 a9b5a7f into master Apr 20, 2026
18 checks passed
@paulineribeyre paulineribeyre deleted the fix-logs branch April 20, 2026 21:40
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