IN-930 - combine unarchiving inputs - Githubissues

eastgenomics / eggd_dias_batch

1 stars 2 forks source link

IN-930 - combine unarchiving inputs #217

Closed jethror1 closed 1 month ago

jethror1 commented 1 month ago

unarchive now always set to True if unarchive_only specified
improve warning message in dx_requests.check_archival_state()

This change is

pep8speaks commented 1 month ago

Hello @jethror1! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file resources/home/dnanexus/dias_batch/dias_batch.py:

Line 176:80: E501 line too long (82 > 79 characters)

In the file resources/home/dnanexus/dias_batch/tests/test_dias_batch.py:

Line 151:46: E712 comparison to True should be 'if cond is True:' or 'if cond:' Line 163:46: E712 comparison to False should be 'if cond is False:' or 'if not cond:'

Comment last updated at 2024-07-25 13:57:29 UTC