Skip to content

Taking out pep8 violations #4

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

Open
lowks wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Taking out pep8 violations #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions parinx/parser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
'x-provider-datacenter': 'datacenter',
}

#FIXME: GK?
# FIXME: GK?
ARGS_TO_XHEADERS_DICT = dict(
([k, v] for v, k in XHEADERS_TO_ARGS_DICT.items()))

Expand Down Expand Up @@ -209,12 +209,12 @@ def parse_docstring(docstring, cls=None):
description, fields_lines = split_docstring(docstring)
arguments_dict = defaultdict(def_arg_dict)
return_value_types = []
#parse fields
# parse fields
return_description = ''
for docstring_line in fields_lines:
if _ignored_field(docstring_line):
continue
#parse inherits
# parse inherits
if docstring_line.startswith('@inherits'):
if not cls:
raise MethodParsingException()
Expand All @@ -227,21 +227,22 @@ def parse_docstring(docstring, cls=None):
arguments_dict[arg_name].update(update_dict)
return_value_types = result['return']['type_name']
return_description = result['return']['description']
#parse return value
# parse return value
elif docstring_line.startswith(':rtype:'):
class_name = __get_class_name(cls)
types_str = docstring_line.split(None, 1)[1]
return_value_types = types_str.replace('\n', '').strip(':').strip()
if return_value_types.startswith('class:`.'):
return_value_types = return_value_types.replace('class:`.', 'class:`'+class_name+'.')
#parse return description
return_value_types = return_value_types.replace(
'class:`.', 'class:`'+class_name+'.')
# parse return description
elif docstring_line.startswith(':return:'):
return_description = docstring_line.split(None, 1)[1].strip()
#parse arguments
# parse argupments
else:
arg_name, update_dict = _parse_docstring_field(cls, docstring_line)
arguments_dict[arg_name].update(update_dict)
#check fields
# check fields
_check_arguments_dict(arguments_dict)
if not return_value_types:
raise MethodParsingException('Can not get return types for method')
Expand Down
2 changes: 1 addition & 1 deletion parinx/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ def default(self, obj):
elif isinstance(obj, datetime.date):
return obj.isoformat()
elif isinstance(obj, datetime.time):
#determines if a given datetime.datetime is aware.
# determines if a given datetime.datetime is aware.
if obj.tzinfo is not None and\
obj.tzinfo.utcoffset(obj) is not None:
raise ValueError("JSON can't represent timezone-aware times.")
Expand Down