Skip to content
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

typing: Account for the fact that requests.HTTPError .response migth be None now #1336

Merged
merged 1 commit into from
Nov 1, 2023

typing: Account for the fact that requests.HTTPError .response migth …

f4a250f
Select commit
Loading
Failed to load commit list.
Merged

typing: Account for the fact that requests.HTTPError .response migth be None now #1336

typing: Account for the fact that requests.HTTPError .response migth …
f4a250f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 1, 2023 in 1s

40.00% of diff hit (target 88.94%)

View this Pull Request on Codecov

40.00% of diff hit (target 88.94%)

Annotations

Check warning on line 168 in dandi/dandiarchive.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dandi/dandiarchive.py#L168 

Added line #L168 was not covered by tests

Check warning on line 300 in dandi/dandiarchive.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dandi/dandiarchive.py#L300 

Added line #L300 was not covered by tests

Check warning on line 703 in dandi/download.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dandi/download.py#L703 

Added line #L703 was not covered by tests
View more details on Codecov