Skip to content

Commit

Permalink
Fixes pytest args, lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
mrharpo committed Aug 3, 2023
1 parent 0de9b20 commit 6d69a45
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ jobs:
uses: WGBH-MLA/.github/.github/workflows/pytest.yml@main
secrets: inherit
with:
pdm_args: -G test,ci
pytest_args: -n auto --nbmake -ra -s
pdm_args: -G test
pytest_args: -v

lint:
name: 👕 Lint
Expand Down
4 changes: 2 additions & 2 deletions fastclam/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ def run_pipeline(pipeline: Pipeline) -> list | dict | str:
log.debug(response.content)
raise HTTPException(
status_code=response.status_code,
detail=f'Unsuccessful response from {app}: {response.status_code} - {response.content}',
detail=f'Unsuccessful response from {app}: {response.status_code} - {response.content}', # noqa E501
)
content_type = response.headers.get('Content-Type')
if 'application/json' in content_type:
Expand All @@ -95,7 +95,7 @@ def run_pipeline(pipeline: Pipeline) -> list | dict | str:
log.debug(response.content)
raise MMIFException(
status_code=499,
detail=f'Failed to parse XML from {app}: {response.status_code} - {response.content}',
detail=f'Failed to parse XML from {app}: {response.status_code} - {response.content}', # noqa E501
)

log.info(f'Ran {len(pipeline.files)} files through {len(pipeline.apps)} apps')
Expand Down
2 changes: 1 addition & 1 deletion fastclam/log.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
handlers=[
{
"sink": RichHandler(),
"format": "<green>{time:YYYY-MM-DD HH:mm:ss.SSS}</green> | <level>{level: <8}</level> | <cyan>{name}</cyan>:<cyan>{function}</cyan>:<cyan>{line}</cyan> - <level>{message}</level>",
"format": "<green>{time:YYYY-MM-DD HH:mm:ss.SSS}</green> | <level>{level: <8}</level> | <cyan>{name}</cyan>:<cyan>{function}</cyan>:<cyan>{line}</cyan> - <level>{message}</level>", # noqa E501
}
]
)

0 comments on commit 6d69a45

Please sign in to comment.