Skip to content

Commit

Permalink
strip newline from version string; fix mypy error
Browse files Browse the repository at this point in the history
  • Loading branch information
jfennick committed Dec 8, 2023
1 parent 77f4959 commit 369c462
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cwltool.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ RUN apk add --no-cache git gcc python3-dev libxml2-dev libxslt-dev libc-dev linu

WORKDIR /cwltool
COPY . .
RUN export SETUPTOOLS_SCM_PRETEND_VERSION_FOR_CWLTOOL=$(grep __version__ cwltool/_version.py | awk -F\' '{ print $2 }') ; \
RUN export SETUPTOOLS_SCM_PRETEND_VERSION_FOR_CWLTOOL=$(grep __version__ cwltool/_version.py | awk -F\' '{ print $2 }' | tr -d '\\n') ; \
CWLTOOL_USE_MYPYC=1 MYPYPATH=mypy-stubs pip wheel --no-binary schema-salad \
--wheel-dir=/wheels .[deps] # --verbose
RUN rm /wheels/schema_salad*
Expand Down
2 changes: 1 addition & 1 deletion cwltool/software_requirements.py
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ def get_container_from_software_requirements(
[DOCKER_CONTAINER_TYPE], tool_info
)
if container_description:
return container_description.identifier
return str(container_description.identifier)

Check warning on line 162 in cwltool/software_requirements.py

View check run for this annotation

Codecov / codecov/patch

cwltool/software_requirements.py#L162

Added line #L162 was not covered by tests

return None

Expand Down

0 comments on commit 369c462

Please sign in to comment.