Run lints

Signed-off-by: GitHub <noreply@github.com>

Author: shenanigansd

events/advent_of_code/2015/03/python/aoc_2015_03.py

@@ -5,7 +5,7 @@
 def part_one(movements: list[str]) -> int:
     delivered_presets = defaultdict(int)
     current_x, current_y = 0, 0
-    delivered_presets[(current_x, current_y)] += 1
+    delivered_presets[current_x, current_y] += 1
     for movement in movements:
         match movement:
             case "^":
@@ -16,15 +16,15 @@ def part_one(movements: list[str]) -> int:
                 current_x -= 1
             case ">":
                 current_x += 1
-        delivered_presets[(current_x, current_y)] += 1
+        delivered_presets[current_x, current_y] += 1
     return len(delivered_presets.keys())
 
 
 def part_two(movements: list[str]) -> int:
     delivered_presets = defaultdict(int)
     santa_x, santa_y = 0, 0
     robot_x, robot_y = 0, 0
-    delivered_presets[(0, 0)] += 2
+    delivered_presets[0, 0] += 2
     for index, movement in enumerate(movements):
         diff_x, diff_y = 0, 0
         match movement:
@@ -39,11 +39,11 @@ def part_two(movements: list[str]) -> int:
         if index % 2 == 0:
             santa_x += diff_x
             santa_y += diff_y
-            delivered_presets[(santa_x, santa_y)] += 1
+            delivered_presets[santa_x, santa_y] += 1
         else:
             robot_x += diff_x
             robot_y += diff_y
-            delivered_presets[(robot_x, robot_y)] += 1
+            delivered_presets[robot_x, robot_y] += 1
     return len(delivered_presets.keys())
 
 

events/advent_of_code/2015/06/python/aoc_2015_06.py

@@ -50,12 +50,12 @@ def part_two(events: list[Event]) -> int:
             for y in range(event.start_y, event.end_y + 1):
                 match event.action:
                     case "on":
-                        brightnesses[(x, y)] += 1
+                        brightnesses[x, y] += 1
                     case "off":
-                        brightnesses[(x, y)] -= 1
-                        brightnesses[(x, y)] = max(brightnesses[(x, y)], 0)
+                        brightnesses[x, y] -= 1
+                        brightnesses[x, y] = max(brightnesses[x, y], 0)
                     case "toggle":
-                        brightnesses[(x, y)] += 2
+                        brightnesses[x, y] += 2
 
     return sum(brightnesses.values())
 

events/advent_of_code/2023/03/python/aoc_2023_03.py

@@ -56,7 +56,7 @@ def find_part_numbers_in_grid(
                             grid[row_index][number_started_on : number_ended_on + 1],
                         ),
                     )
-                    part_numbers[(number, row_index, number_started_on)] = find_adjacent_symbol(
+                    part_numbers[number, row_index, number_started_on] = find_adjacent_symbol(
                         grid,
                         row_index,
                         number_started_on,

events/advent_of_code/2024/01/python/aoc_2024_01.py

@@ -4,7 +4,7 @@
 
 def parse_input(text: str) -> tuple[list[int], list[int]]:
     rows = text.strip().split("\n")
-    
+
     left = []
     right = []
     for row in rows:
@@ -28,4 +28,4 @@ def part2(left: list[int], right: list[int]) -> int:
 if __name__ == "__main__":
     rows = Path("../input.txt").read_text(encoding="locale")
     print(part1(*parse_input(rows)))
-    print(part2(*parse_input(rows)))
+    print(part2(*parse_input(rows)))

events/advent_of_code/2024/01/python/test_aoc_2024_01.py

@@ -1,4 +1,3 @@
-import pytest
 from aoc_2024_01 import parse_input, part1, part2
 
 TEST_INPUT = """
@@ -10,11 +9,14 @@
 3   3
 """
 
+
 def test_parse_input() -> None:
     assert parse_input(TEST_INPUT) == ([1, 2, 3, 3, 3, 4], [3, 3, 3, 4, 5, 9])
 
+
 def test_part1() -> None:
     assert part1(*parse_input(TEST_INPUT)) == 11
 
+
 def test_part2() -> None:
     assert part2(*parse_input(TEST_INPUT)) == 31

pyproject.toml

@@ -66,11 +66,18 @@ ignore = [
   "D103", # (Missing docstring in public function) - Not really doc-ing here
   "D104", # (Missing docstring in public package) - Not really doc-ing here
   "D105", # (Missing docstring in magic method) - Not really doc-ing here
+  "DOC", # (pydoclint) - Not really doc-ing here
   "S311", # (Standard pseudo-random generators are not suitable for security/cryptographic purposes) - Not doing crypto here
   "PLR2004", # (Magic value used in comparison) - Lots of single use numbers in AoC
   "PLC2701", # (Private name import `x` from external module `x`) - Testing internal methods in AoC
   "C901", # (Function is too complex) - I completely agree, but it's AoC
 ]
 
+[tool.ruff.lint.extend-per-file-ignores]
+"scratch/games/the-farmer-was-replaced/code.py" = [
+    "F821", # (Undefined name `x`)
+    "ANN001", # (Missing type annotation for function argument `x`)
+]
+
 [tool.ruff.lint.pydocstyle]
 convention = "numpy"

scratch/games/the-farmer-was-replaced/code.py

@@ -1,4 +1,4 @@
-def move_to_pos(x, y):
+def move_to_pos(x, y) -> None:
     current_x = get_pos_x()
     current_y = get_pos_y()
 
@@ -26,62 +26,55 @@ def move_to_pos(x, y):
 move_to_pos(0, 0)
 
 
-def odd(a):
-    if a % 2 == 0:
-        return False
-    else:
-        return True
+def odd(a) -> bool:
+    return a % 2 != 0
 
 
-def even(a):
-    if a % 2 == 0:
-        return True
-    else:
-        return False
+def even(a) -> bool:
+    return a % 2 == 0
 
 
-def tplace():
-    if odd(get_pos_x()) == False and odd(get_pos_y()) == False:
+def tplace() -> bool | None:
+    if odd(get_pos_x()) is False and odd(get_pos_y()) is False:
         return True
-    elif odd(get_pos_x()) == True and odd(get_pos_y()) == True:
+    if odd(get_pos_x()) is True and odd(get_pos_y()) is True:
         return True
+    return None
 
 
 while True:
-    for i in range(get_world_size()):
+    for _i in range(get_world_size()):
         if get_pos_y() == 0:
             plant(Entities.grass)
             if can_harvest():
                 harvest()
 
-        else:
-            if tplace() == True:
-                if get_ground_type() == Grounds.Soil:
-                    pass
-                else:
-                    till()
+        elif tplace() is True:
+            if get_ground_type() == Grounds.Soil:
+                pass
+            else:
+                till()
 
-                plant(Entities.Tree)
+            plant(Entities.Tree)
 
-                if get_water() < 0.5:
-                    use_item(Items.Water_Tank)
+            if get_water() < 0.5:
+                use_item(Items.Water_Tank)
 
-            else:
-                if (
-                    get_ground_type() == Grounds.soil
-                    and get_entity_type() == Entities.Carrots
-                    and can_harvest() == True
-                ):
-                    harvest()
-                    trade(Items.Carrot_Seed)
-                    plant(Entities.Carrots)
-                elif get_ground_type() != Grounds.Soil:
-                    till()
-                    trade(Items.Carrot_Seed)
-                    plant(Entities.Carrots)
-                elif get_ground_type() == Grounds.Soil and get_entity_type() != Entities.Carrots:
-                    trade(Items.Carrot_Seed)
-                    plant(Entities.Carrots)
+        elif (
+            get_ground_type() == Grounds.soil
+            and get_entity_type() == Entities.Carrots
+            and can_harvest() is True
+        ):
+            harvest()
+            trade(Items.Carrot_Seed)
+            plant(Entities.Carrots)
+        elif get_ground_type() != Grounds.Soil:
+            till()
+            trade(Items.Carrot_Seed)
+            plant(Entities.Carrots)
+        elif get_ground_type() == Grounds.Soil and get_entity_type() != Entities.Carrots:
+            trade(Items.Carrot_Seed)
+            plant(Entities.Carrots)
         move(East)
         if can_harvest():
             harvest()