Skip to content

Commit

Permalink
Merge branch 'release_24.1' into dev
Browse files Browse the repository at this point in the history
  • Loading branch information
nsoranzo committed Oct 9, 2024
2 parents dc8a56b + 80a99f5 commit 01512cb
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 14 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,4 +55,6 @@ jobs:
- name: Run mypy checks
run: tox -e mypy
- uses: psf/black@stable
with:
version: "24.8.0" # last version supporting Python 3.8
- uses: isort/isort-action@v1
6 changes: 1 addition & 5 deletions lib/galaxy/tool_util/cwl/__init__.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@
from .cwltool_deps import (
needs_shell_quoting,
shellescape,
)
from .cwltool_deps import needs_shell_quoting
from .parser import (
tool_proxy,
tool_proxy_from_persistent_representation,
Expand All @@ -21,5 +18,4 @@
"to_cwl_job",
"to_galaxy_parameters",
"needs_shell_quoting",
"shellescape",
)
10 changes: 1 addition & 9 deletions lib/galaxy/tool_util/cwl/cwltool_deps.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,11 +71,6 @@
visit_class = None # type: ignore[assignment]
normalizeFilesDirs = None # type: ignore[assignment]

try:
import shellescape
except ImportError:
shellescape = None

try:
import schema_salad
from schema_salad import (
Expand Down Expand Up @@ -105,16 +100,14 @@ def ensure_cwltool_available():
Throw an ImportError with a description of the problem if they do not exist.
"""
if main is None or workflow is None or shellescape is None:
if main is None or workflow is None:
message = "This feature requires cwltool and dependencies to be available, they are not."
if main is None:
message += " cwltool is not unavailable."
elif resolve_and_validate_document is None:
message += " cwltool.load_tool.resolve_and_validate_document is unavailable - cwltool version is too old."
if requests is None:
message += " Library 'requests' unavailable."
if shellescape is None:
message += " Library 'shellescape' unavailable."
if schema_salad is None:
message += " Library 'schema_salad' unavailable."
raise ImportError(message)
Expand All @@ -137,7 +130,6 @@ def ensure_cwltool_available():
"resolve_and_validate_document",
"RuntimeContext",
"schema_salad",
"shellescape",
"sourceline",
"StdFsAccess",
"visit_class",
Expand Down

0 comments on commit 01512cb

Please sign in to comment.