-
Notifications
You must be signed in to change notification settings - Fork 15
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.
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
Fix general PEP8 problems, improve maintainability #67
Conversation
pynytimes/article_search.py
Outdated
NoneType = type(None) | ||
|
||
|
||
def article_search_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function article_search_check_input
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
pynytimes/article_search.py
Outdated
return datetime.datetime(input.year, input.month, input.day) | ||
|
||
|
||
def article_search_parse_dates( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function article_search_parse_dates
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
pynytimes/movie_reviews.py
Outdated
return datetime.datetime(input.year, input.month, input.day) | ||
|
||
|
||
def movie_reviews_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function movie_reviews_check_input
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
pynytimes/movie_reviews.py
Outdated
raise TypeError("Critics Pick needs to be a bool") | ||
|
||
|
||
def movie_reviews_parse_dates( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function movie_reviews_parse_dates
has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.
pynytimes/book_reviews.py
Outdated
NoneType: type = type(None) | ||
|
||
|
||
def book_reviews_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function book_reviews_check_input
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
pynytimes/helpers/article_search.py
Outdated
NoneType = type(None) | ||
|
||
|
||
def article_search_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function article_search_check_input
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
return datetime.datetime(input.year, input.month, input.day) | ||
|
||
|
||
def article_search_parse_dates( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function article_search_parse_dates
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
pynytimes/helpers/movie_reviews.py
Outdated
return datetime.datetime(input.year, input.month, input.day) | ||
|
||
|
||
def movie_reviews_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function movie_reviews_check_input
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
raise TypeError("Critics Pick needs to be a bool") | ||
|
||
|
||
def movie_reviews_parse_dates( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function movie_reviews_parse_dates
has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.
NoneType: type = type(None) | ||
|
||
|
||
def book_reviews_check_input( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function book_reviews_check_input
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
No description provided.