Skip to content

Commit

Permalink
Revert isinstance(step_entry, (str, unicode)) because typing.Text is …
Browse files Browse the repository at this point in the history
…actually

<type 'unicode'> and as a result isinstance(str("foo"), Text) is False.
  • Loading branch information
Peter Amstutz committed Aug 29, 2016
1 parent 0433da3 commit 6a8d5f4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cwltool/workflow.py
Original file line number Diff line number Diff line change
Expand Up @@ -447,7 +447,7 @@ def __init__(self, toolpath_object, pos, **kwargs):
for stepfield, toolfield in (("in", "inputs"), ("out", "outputs")):
toolpath_object[toolfield] = []
for step_entry in toolpath_object[stepfield]:
if isinstance(step_entry, (Text, Text)):
if isinstance(step_entry, (str, unicode)):
param = {} # type: Dict[Text, Any]
inputid = step_entry
else:
Expand Down

1 comment on commit 6a8d5f4

@mr-c
Copy link
Member

@mr-c mr-c commented on 6a8d5f4 Aug 30, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should have been (str, Text) as Text is str under Python3

Please sign in to comment.