flake8

Author: ikostan

.flake8

@@ -3,8 +3,10 @@
 # https://flake8.pycqa.org/en/3.1.1/user/configuration.html#configuration
 # Errors to exclude:
 # W191 indentation contains tabs
-ignore = W191, W503
+ignore = W191, W503, E203
 exclude =
+    solutions/,
+    *_test.py,
     .git,
     __pycache__,
     .circleci,
@@ -21,5 +23,4 @@ exclude =
     /.venv/,
     /.venv,
     migrations,
-max-complexity = 10
-max-line-length = 145
+max-complexity = 10

.github/workflows/flake8.yml

@@ -60,4 +60,4 @@ jobs:
         # Changing the Ignore List
         # W191 indentation contains tabs
         run: |
-          python -m flake8 . --count --select=E9,F63,F7,F82 --doctests --show-source --statistics
+          python -m flake8 . --count --doctests --show-source --statistics

bob/bob.py

@@ -1,23 +1,27 @@
 """
-Your task is to determine what Bob will reply to someone when they say something
-to him or ask him a question.
+Your task is to determine what Bob will reply to someone when
+they say something to him or ask him a question.
 
 Bob only ever answers one of five things:
 
-- "Sure." This is his response if you ask him a question, such as "How are you?"
+- "Sure." This is his response if you ask him a question,
+  such as "How are you?"
   The convention used for questions is that it ends with a question mark.
 - "Whoa, chill out!" This is his answer if you YELL AT HIM.
   The convention used for yelling is ALL CAPITAL LETTERS.
-- "Calm down, I know what I'm doing!" This is what he says if you yell a question at him.
+- "Calm down, I know what I'm doing!" This is what he says if you
+   yell a question at him.
 - "Fine. Be that way!" This is how he responds to silence.
-  The convention used for silence is nothing, or various combinations of whitespace characters.
+  The convention used for silence is nothing, or various combinations
+  of whitespace characters.
 - "Whatever." This is what he answers to anything else.
 """
 
 
 def response(hey_bob: str) -> str:
     """
-    Determine what Bob will reply to someone when they say something to him or ask him a question.
+    Determine what Bob will reply to someone when they say
+    something to him or ask him a question.
 
     :param hey_bob: str
     :return: str

card-games/lists.py

@@ -1,7 +1,8 @@
 """
 Functions for tracking poker hands and assorted card tasks.
 
-Python list documentation: https://docs.python.org/3/tutorial/datastructures.html
+Python list documentation:
+https://docs.python.org/3/tutorial/datastructures.html
 """
 
 
@@ -49,18 +50,22 @@ def card_average(hand: list[list]) -> float:
 
 def approx_average_is_average(hand: list[int]) -> bool:
     """
-    Return if the (average of first and last card values) OR ('middle' card) == calculated average.
+    Return if the (average of first and last card values) OR
+    ('middle' card) == calculated average.
 
     :param hand: list - cards in hand.
-    :return: bool - does one of the approximate averages equal the `true average`?
+    :return: bool - does one of the approximate averages
+             equal the `true average`?
     """
     avg: float = card_average(hand)
     return avg in ((hand[0] + hand[-1]) / 2, hand[len(hand) // 2])
 
 
 def average_even_is_average_odd(hand: list[int]) -> bool:
     """
-    Return if the (average of even indexed card values) == (average of odd indexed card values).
+    Return if the
+    (average of even indexed card values) ==
+    (average of odd indexed card values).
 
     :param hand: list - cards in hand.
     :return: bool - are even and odd averages equal?

chaitanas-colossal-coaster/list_methods.py

@@ -8,7 +8,8 @@ def add_me_to_the_queue(
     person_name: str,
 ) -> list[str]:
     """
-    Add a person to the 'express' or 'normal' queue depending on the ticket number.
+    Add a person to the 'express' or 'normal' queue depending
+    on the ticket number.
 
     :param express_queue: list[str] - names in the Fast-track queue.
     :param normal_queue: list[str] - names in the normal queue.
@@ -35,7 +36,9 @@ def find_my_friend(queue: list[str], friend_name: str) -> int:
     return queue.index(friend_name)
 
 
-def add_me_with_my_friends(queue: list[str], index: int, person_name: str) -> list[str]:
+def add_me_with_my_friends(
+    queue: list[str], index: int, person_name: str
+) -> list[str]:
     """
     Insert the late arrival's name at a specific index of the queue.
 

currency-exchange/exchange.py

@@ -28,7 +28,8 @@ def get_change(budget: float, exchanging_value: float) -> float:
     Return the amount of money that "is left" from the budget.
 
     :param budget: float - amount of money you own.
-    :param exchanging_value: float - amount of your money you want to exchange now.
+    :param exchanging_value: float - amount of your money you want to
+                             exchange now.
     :return: float - amount left of your starting currency after exchanging.
     """
     return budget - exchanging_value  # pylint: disable=R0801
@@ -51,7 +52,8 @@ def get_value_of_bills(denomination: float, number_of_bills: float) -> float:
 
 def get_number_of_bills(amount: float, denomination: int) -> int:
     """
-    Return the _number of currency bills_ that you can receive within the given _amount_.
+    Return the _number of currency bills_ that you can receive within
+    the given _amount_.
 
     :param amount: float - the total starting value.
     :param denomination: int - the value of a single bill.
@@ -62,12 +64,13 @@ def get_number_of_bills(amount: float, denomination: int) -> int:
 
 def get_leftover_of_bills(amount: float, denomination: int) -> float:
     """
-    Return the _leftover amount_ that cannot be returned from your starting _amount_
-    given the denomination of bills.
+    Return the _leftover amount_ that cannot be returned from your
+    starting _amount_ given the denomination of bills.
 
     :param amount: float - the total starting value.
     :param denomination: int - the value of a single bill.
-    :return: float - the amount that is "leftover", given the current denomination.
+    :return: float - the amount that is "leftover", given the current
+             denomination.
     """
     return amount % denomination
 
@@ -80,7 +83,8 @@ def exchangeable_value(
     Return the maximum value of the new currency after calculating
     the *exchange rate* plus the *spread*.
 
-    :param budget: float - the amount of your money you are planning to exchange.
+    :param budget: float - the amount of your money you are planning
+                   to exchange.
     :param exchange_rate: float - the unit value of the foreign currency.
     :param spread: int - percentage that is taken as an exchange fee.
     :param denomination: int - the value of a single bill.

ghost-gobble-arcade-game/arcade_game.py

@@ -5,7 +5,8 @@ def eat_ghost(power_pellet_active: bool, touching_ghost: bool) -> bool:
     """
     Verify that Pac-Man can eat a ghost if he is empowered by a power pellet.
 
-    :param power_pellet_active: bool - does the player have an active power pellet?
+    :param power_pellet_active: bool - does the player have an active
+                                power pellet?
     :param touching_ghost: bool - is the player touching a ghost?
     :return: bool - can a ghost be eaten?
     """
@@ -25,9 +26,11 @@ def score(touching_power_pellet: bool, touching_dot: bool) -> bool:
 
 def lose(power_pellet_active: bool, touching_ghost: bool) -> bool:
     """
-    Trigger the game loop to end (GAME OVER) when Pac-Man touches a ghost without his power pellet.
+    Trigger the game loop to end (GAME OVER) when Pac-Man touches
+    a ghost without his power pellet.
 
-    :param power_pellet_active: bool - does the player have an active power pellet?
+    :param power_pellet_active: bool - does the player have an
+           active power pellet?
     :param touching_ghost: bool - is the player touching a ghost?
     :return: bool - has the player lost the game?
     """
@@ -41,7 +44,8 @@ def win(
     Trigger the victory event when all dots have been eaten.
 
     :param has_eaten_all_dots: bool - has the player "eaten" all the dots?
-    :param power_pellet_active: bool - does the player have an active power pellet?
+    :param power_pellet_active: bool - does the player have an active
+           power pellet?
     :param touching_ghost: bool - is the player touching a ghost?
     :return: bool - has the player won the game?
     """

guidos-gorgeous-lasagna/lasagna.py

@@ -22,7 +22,8 @@ def bake_time_remaining(elapsed_bake_time: int) -> int:
     based on the `EXPECTED_BAKE_TIME`.
 
     :param elapsed_bake_time: int - baking time already elapsed.
-    :return: int - remaining bake time (in minutes) derived from 'EXPECTED_BAKE_TIME'.
+    :return: int - remaining bake time (in minutes) derived from
+             'EXPECTED_BAKE_TIME'.
     """
     return EXPECTED_BAKE_TIME - elapsed_bake_time
 
@@ -31,8 +32,8 @@ def preparation_time_in_minutes(number_of_layers: int) -> int:
     """
     Calculate preparation time in minutes
 
-    Takes the `number_of_layers` you want to add to the lasagna as an argument and
-    returns how many minutes you would spend making them.
+    Takes the `number_of_layers` you want to add to the lasagna as an
+    argument and returns how many minutes you would spend making them.
 
     Assume each layer takes 2 minutes to prepare.
 
@@ -44,12 +45,15 @@ def preparation_time_in_minutes(number_of_layers: int) -> int:
     return int(PREPARATION_TIME * number_of_layers)
 
 
-def elapsed_time_in_minutes(number_of_layers: int, elapsed_bake_time: int) -> int:
+def elapsed_time_in_minutes(
+    number_of_layers: int, elapsed_bake_time: int
+) -> int:
     """
     Calculate elapsed time in minutes.
 
-    Return the total minutes you have been in the kitchen cooking — your preparation time layering +
-    the time the lasagna has spent baking in the oven.
+    Return the total minutes you have been in the kitchen cooking — your
+    preparation time layering + the time the lasagna has spent baking
+    in the oven.
 
     :param number_of_layers: The number of layers added to the lasagna.
     :type number_of_layers: int

little-sisters-vocab/strings.py

@@ -13,8 +13,8 @@ def add_prefix_un(word: str) -> str:
 
 def make_word_groups(vocab_words: list[str]) -> str:
     """
-    Transform a list containing a prefix and words into a string with the prefix
-    followed by the words with prefix prepended.
+    Transform a list containing a prefix and words into a string with
+    the prefix followed by the words with prefix prepended.
 
     :param vocab_words: list - of vocabulary words with prefix in first index.
     :return: str - of prefix followed by vocabulary words with

making-the-grade/loops.py

@@ -18,7 +18,8 @@ def count_failed_students(student_scores: list) -> int:
     :param student_scores: list - containing int student scores.
     :return: int - count of student scores at or below 40.
     """
-    return len([score for score in student_scores if score <= 40.0])  # pylint: disable=R0801
+    # pylint: disable=R0801
+    return len([score for score in student_scores if score <= 40.0])
 
 
 def above_threshold(student_scores: list, threshold: int) -> list:
@@ -30,19 +31,21 @@ def above_threshold(student_scores: list, threshold: int) -> list:
 
     :param student_scores: list - of integer scores.
     :param threshold: int - threshold to cross to be the "best" score.
-    :return: list - of integer scores that are at or above the "best" threshold.
+    :return: list - of integer scores that are at or above the "best"
+             threshold.
     """
-    return [score for score in student_scores if score >= threshold]  # pylint: disable=R0801
+    # pylint: disable=R0801
+    return [score for score in student_scores if score >= threshold]
 
 
 def letter_grades(highest: int) -> list:
     """
     Create a list of grade thresholds based on the provided highest grade.
 
     :param highest: int - value of highest exam score.
-    :return: list - of lower threshold scores for each D-A letter grade interval.
-            For example, where the highest score is 100, and failing is <= 40,
-            The result would be [41, 56, 71, 86]:
+    :return: list - of lower threshold scores for each D-A letter grade
+            interval. For example, where the highest score is 100, and
+            failing is <= 40, The result would be [41, 56, 71, 86]:
 
             41 <= "D" <= 55
             56 <= "C" <= 70
@@ -56,10 +59,12 @@ def letter_grades(highest: int) -> list:
 # pylint: disable=R0801
 def student_ranking(student_scores: list, student_names: list) -> list[str]:
     """
-    Organize the student's rank, name, and grade information in descending order.
+    Organize the student's rank, name, and grade information in descending
+    order.
 
     :param student_scores: list - of scores in descending order.
-    :param student_names: list - of string names by exam score in descending order.
+    :param student_names: list - of string names by exam score in descending
+                          order.
     :return: list - of strings in format ["<rank>. <student name>: <score>"].
     """
     return [