Fix PEP8 max line length violation

close #434

Author: behrtam

.travis.yml

@@ -18,7 +18,7 @@ install:
   - pip install -r requirements-travis.txt
 
 before_script:
-  - flake8 ./exercises/ --max-line-length=99 --select=E,W
+  - flake8 .
 
 script:
   - ./test/check-exercises.py

README.md

@@ -35,11 +35,7 @@ python test/check-exercises.py
 
 The Python code in this repo is meant to largely obey the [PEP8 style guide](https://www.python.org/dev/peps/pep-0008/).
 
-This repo uses [flake8](http://flake8.readthedocs.org/en/latest/) to enforce the coding standard. When you submit a PR, it needs to pass the flake8 tool with no warnings, or it won't be accepted. Here are the settings used by the build system:
-
-```
-flake8 [your-code-here.py] --max-line-length=99 --select=E,W
-```
+This repo uses [flake8](http://flake8.readthedocs.org/en/latest/) to enforce the coding standard. When you submit a PR, it needs to pass the flake8 tool with no warnings, or it won't be accepted.
 
 ## Pull Requests
 

exercises/accumulate/accumulate_test.py

@@ -8,8 +8,8 @@ def test_empty_sequence(self):
         self.assertEqual([], accumulate([], lambda x: x / 2))
 
     def test_pow(self):
-        self.assertEqual([1, 4, 9, 16, 25], accumulate([1, 2, 3, 4, 5],
-                         lambda x: x * x))
+        self.assertEqual([1, 4, 9, 16, 25],
+                         accumulate([1, 2, 3, 4, 5], lambda x: x * x))
 
     def test_divmod(self):
         inp = [10, 17, 23]
@@ -18,8 +18,10 @@ def test_divmod(self):
 
     def test_composition(self):
         inp = [10, 17, 23]
-        self.assertEqual(inp, accumulate(accumulate(inp, lambda x: divmod(x, 7)),
-                         lambda x: 7 * x[0] + x[1]))
+        self.assertEqual(inp,
+                         accumulate(
+                             accumulate(inp, lambda x: divmod(x, 7)),
+                             lambda x: 7 * x[0] + x[1]))
 
     def test_capitalize(self):
         inp = ['hello', 'world']
@@ -29,8 +31,10 @@ def test_capitalize(self):
     def test_recursive(self):
         inp = list('abc')
         out = [['a1', 'a2', 'a3'], ['b1', 'b2', 'b3'], ['c1', 'c2', 'c3']]
-        self.assertEqual(out, accumulate(inp, lambda x: accumulate(list('123'),
-                         lambda y: x + y)))
+        self.assertEqual(
+            out,
+            accumulate(inp,
+                       lambda x: accumulate(list('123'), lambda y: x + y)))
 
 
 if __name__ == '__main__':

exercises/acronym/acronym_test.py

@@ -2,9 +2,9 @@
 
 from acronym import abbreviate
 
-
 # test cases adapted from `x-common//canonical-data.json` @ version: 1.0.0
 
+
 class AcronymTest(unittest.TestCase):
     def test_basic(self):
         self.assertEqual('PNG', abbreviate('Portable Network Graphics'))
@@ -25,7 +25,8 @@ def test_non_acronym_all_caps_word(self):
         self.assertEqual('GIMP', abbreviate('GNU Image Manipulation Program'))
 
     def test_hyphenated(self):
-        self.assertEqual('CMOS', abbreviate('Complementary metal-oxide semiconductor'))
+        self.assertEqual('CMOS',
+                         abbreviate('Complementary metal-oxide semiconductor'))
 
 
 if __name__ == '__main__':

exercises/all-your-base/all_your_base_test.py

@@ -2,11 +2,10 @@
 
 from all_your_base import rebase
 
-
 # test cases adapted from `x-common//canonical-data.json` @ version: 1.0.0
 
-class AllYourBaseTests(unittest.TestCase):
 
+class AllYourBaseTests(unittest.TestCase):
     def test_single_bit_to_one_decimal(self):
         self.assertEqual(rebase(2, [1], 10), [1])
 

exercises/all-your-base/example.py

@@ -1,5 +1,5 @@
 def from_digits(digits, base):
-    return sum(n * base ** i for i, n in enumerate(reversed(digits)))
+    return sum(n * base**i for i, n in enumerate(reversed(digits)))
 
 
 def to_digits(number, base_to):

exercises/allergies/allergies_test.py

@@ -4,7 +4,6 @@
 
 
 class AllergiesTests(unittest.TestCase):
-
     def test_no_allergies_means_not_allergic(self):
         allergies = Allergies(0)
         self.assertFalse(allergies.is_allergic_to('peanuts'))

exercises/allergies/example.py

@@ -1,14 +1,8 @@
 class Allergies(object):
 
     _allergies = [
-        "eggs",
-        "peanuts",
-        "shellfish",
-        "strawberries",
-        "tomatoes",
-        "chocolate",
-        "pollen",
-        "cats"
+        "eggs", "peanuts", "shellfish", "strawberries", "tomatoes",
+        "chocolate", "pollen", "cats"
     ]
 
     def __init__(self, score):
@@ -19,5 +13,7 @@ def is_allergic_to(self, allergy):
 
     @property
     def lst(self):
-        return [allergy for allergy in self._allergies
-                if self.is_allergic_to(allergy)]
+        return [
+            allergy for allergy in self._allergies
+            if self.is_allergic_to(allergy)
+        ]

exercises/anagram/anagram_test.py

@@ -6,66 +6,45 @@
 class AnagramTests(unittest.TestCase):
     def test_no_matches(self):
         self.assertEqual(
-            [],
-            detect_anagrams('diaper', 'hello world zombies pants'.split())
-        )
+            [], detect_anagrams('diaper', 'hello world zombies pants'.split()))
 
     def test_detect_simple_anagram(self):
-        self.assertEqual(
-            ['tan'],
-            detect_anagrams('ant', 'tan stand at'.split())
-        )
+        self.assertEqual(['tan'],
+                         detect_anagrams('ant', 'tan stand at'.split()))
 
     def test_detect_multiple_anagrams(self):
         self.assertEqual(
             ['stream', 'maters'],
-            detect_anagrams('master', 'stream pigeon maters'.split())
-        )
+            detect_anagrams('master', 'stream pigeon maters'.split()))
 
     def test_does_not_confuse_different_duplicates(self):
-        self.assertEqual(
-            [],
-            detect_anagrams('galea', ['eagle'])
-        )
+        self.assertEqual([], detect_anagrams('galea', ['eagle']))
 
     def test_eliminate_anagram_subsets(self):
-        self.assertEqual(
-            [],
-            detect_anagrams('good', 'dog goody'.split())
-        )
+        self.assertEqual([], detect_anagrams('good', 'dog goody'.split()))
 
     def test_detect_anagram(self):
         self.assertEqual(
             ['inlets'],
-            detect_anagrams('listen', 'enlists google inlets banana'.split())
-        )
+            detect_anagrams('listen', 'enlists google inlets banana'.split()))
 
     def test_multiple_anagrams(self):
         self.assertEqual(
             'gallery regally largely'.split(),
             detect_anagrams(
                 'allergy',
-                'gallery ballerina regally clergy largely leading'.split()
-            )
-        )
+                'gallery ballerina regally clergy largely leading'.split()))
 
     def test_anagrams_are_case_insensitive(self):
         self.assertEqual(
             ['Carthorse'],
             detect_anagrams('Orchestra',
-                            'cashregister Carthorse radishes'.split())
-        )
+                            'cashregister Carthorse radishes'.split()))
 
     def test_same_word_isnt_anagram(self):
-        self.assertEqual(
-            [],
-            detect_anagrams('banana', ['banana'])
-        )
+        self.assertEqual([], detect_anagrams('banana', ['banana']))
 
-        self.assertEqual(
-            [],
-            detect_anagrams('go', 'go Go GO'.split())
-        )
+        self.assertEqual([], detect_anagrams('go', 'go Go GO'.split()))
 
 
 if __name__ == '__main__':

exercises/anagram/example.py

@@ -1,8 +1,9 @@
 def detect_anagrams(word, candidates):
-    return [candidate
-            for candidate in candidates
-            if _letters(candidate) == _letters(word)
-            if candidate.lower() != word.lower()]
+    return [
+        candidate for candidate in candidates
+        if _letters(candidate) == _letters(word)
+        if candidate.lower() != word.lower()
+    ]
 
 
 def _letters(word):

exercises/atbash-cipher/atbash_cipher_test.py

@@ -2,11 +2,10 @@
 
 from atbash_cipher import decode, encode
 
-
 # test cases adapted from `x-common//canonical-data.json` @ version: 1.0.0
 
-class AtbashCipherTest(unittest.TestCase):
 
+class AtbashCipherTest(unittest.TestCase):
     def test_encode_no(self):
         self.assertMultiLineEqual("ml", encode("no"))
 
@@ -41,14 +40,11 @@ def test_decode_word(self):
     def test_decode_sentence(self):
         self.assertMultiLineEqual(
             "anobstacleisoftenasteppingstone",
-            decode("zmlyh gzxov rhlug vmzhg vkkrm thglm v")
-        )
+            decode("zmlyh gzxov rhlug vmzhg vkkrm thglm v"))
 
     def test_decode_numbers(self):
-        self.assertMultiLineEqual(
-            "testing123testing",
-            decode("gvhgr mt123 gvhgr mt")
-        )
+        self.assertMultiLineEqual("testing123testing",
+                                  decode("gvhgr mt123 gvhgr mt"))
 
     def test_decode_all_the_letters(self):
         ciphertext = "gsvjf rxpyi ldmul cqfnk hlevi gsvoz abwlt"
@@ -57,10 +53,8 @@ def test_decode_all_the_letters(self):
 
     # additional track specific test
     def test_encode_decode(self):
-        self.assertMultiLineEqual(
-            "testing123testing",
-            decode(encode("Testing, 1 2 3, testing."))
-        )
+        self.assertMultiLineEqual("testing123testing",
+                                  decode(encode("Testing, 1 2 3, testing.")))
 
 
 if __name__ == '__main__':

exercises/atbash-cipher/example.py

@@ -6,7 +6,6 @@
 else:
     maketrans = str.maketrans
 
-
 BLKSZ = 5
 trtbl = maketrans(ascii_lowercase, ascii_lowercase[::-1])
 
@@ -17,8 +16,8 @@ def base_trans(text):
 
 def encode(plain):
     cipher = base_trans(plain)
-    return " ".join([cipher[i:i + BLKSZ]
-                     for i in range(0, len(cipher), BLKSZ)])
+    return " ".join(
+        [cipher[i:i + BLKSZ] for i in range(0, len(cipher), BLKSZ)])
 
 
 def decode(ciphered):

exercises/beer-song/beer_song_test.py

@@ -6,24 +6,19 @@
 class BeerTest(unittest.TestCase):
     def test_a_verse(self):
         self.assertEqual(
-            verse(8),
-            "8 bottles of beer on the wall, 8 bottles of beer.\n"
+            verse(8), "8 bottles of beer on the wall, 8 bottles of beer.\n"
             "Take one down and pass it around, "
-            "7 bottles of beer on the wall.\n"
-        )
+            "7 bottles of beer on the wall.\n")
 
     def test_verse_1(self):
         self.assertEqual(
-            verse(1),
-            "1 bottle of beer on the wall, 1 bottle of beer.\n"
+            verse(1), "1 bottle of beer on the wall, 1 bottle of beer.\n"
             "Take it down and pass it around, "
-            "no more bottles of beer on the wall.\n"
-        )
+            "no more bottles of beer on the wall.\n")
 
     def test_verse_2(self):
         self.assertEqual(
-            verse(2),
-            "2 bottles of beer on the wall, 2 bottles of beer.\n"
+            verse(2), "2 bottles of beer on the wall, 2 bottles of beer.\n"
             "Take one down and pass it around, 1 bottle of beer on the wall.\n"
         )
 
@@ -32,27 +27,23 @@ def test_verse_0(self):
             verse(0),
             "No more bottles of beer on the wall, no more bottles of beer.\n"
             "Go to the store and buy some more, "
-            "99 bottles of beer on the wall.\n"
-        )
+            "99 bottles of beer on the wall.\n")
 
     def test_songing_several_verses(self):
         self.assertEqual(
-            song(8, 6),
-            "8 bottles of beer on the wall, 8 bottles of beer.\n"
+            song(8, 6), "8 bottles of beer on the wall, 8 bottles of beer.\n"
             "Take one down and pass it around, "
             "7 bottles of beer on the wall.\n\n"
             "7 bottles of beer on the wall, 7 bottles of beer.\n"
             "Take one down and pass it around, "
             "6 bottles of beer on the wall.\n\n"
             "6 bottles of beer on the wall, 6 bottles of beer.\n"
             "Take one down and pass it around, "
-            "5 bottles of beer on the wall.\n\n"
-        )
+            "5 bottles of beer on the wall.\n\n")
 
     def test_song_all_the_rest_of_the_verses(self):
         self.assertEqual(
-            song(3),
-            "3 bottles of beer on the wall, 3 bottles of beer.\n"
+            song(3), "3 bottles of beer on the wall, 3 bottles of beer.\n"
             "Take one down and pass it around, "
             "2 bottles of beer on the wall.\n\n"
             "2 bottles of beer on the wall, 2 bottles of beer.\n"
@@ -63,8 +54,7 @@ def test_song_all_the_rest_of_the_verses(self):
             "no more bottles of beer on the wall.\n\n"
             "No more bottles of beer on the wall, no more bottles of beer.\n"
             "Go to the store and buy some more, "
-            "99 bottles of beer on the wall.\n\n"
-        )
+            "99 bottles of beer on the wall.\n\n")
 
 
 if __name__ == '__main__':

exercises/beer-song/example.py

@@ -19,9 +19,9 @@ def _action(current_verse):
     if current_verse == 0:
         return "Go to the store and buy some more, "
     else:
-        return "Take %s down and pass it around, " % (
-            "one" if current_verse > 1 else "it"
-        )
+        return "Take %s down and pass it around, " % ("one"
+                                                      if current_verse > 1 else
+                                                      "it")
 
 
 def _next_bottle(current_verse):