Fixed linting

Author: smangham

.idea/misc.xml

@@ -18,15 +18,15 @@ install:  ## install library
 .PHONY: lint lints fix format
 
 lint:  ## run python linter with ruff
-	python -m ruff check physics-workload app core users
-	python -m ruff format --check physics-workload app core users
+	python -m ruff check physics_workload
+	python -m ruff format --check physics_workload
 
 # Alias
 lints: lint
 
 fix:  ## fix python formatting with ruff
-	python -m ruff check --fix physics-workload app core users
-	python -m ruff format physics-workload app core users
+	python -m ruff check --fix physics_workload
+	python -m ruff format physics_workload
 
 # alias
 format: fix

.idea/physics-workload.iml

@@ -45,7 +45,7 @@ class Task(ModelCommon):
 
     # === CHANGEABLE FIELDS ===
     name = CharField(max_length=128, blank=False)  # A hidden, uneditable, qualified name
-    title = CharField(max_length=128, blank=False)   # The editable version of the name
+    title = CharField(max_length=128, blank=False)  # The editable version of the name
 
     is_required = BooleanField(
         default=False,
@@ -146,9 +146,7 @@ class Task(ModelCommon):
     # ==========================================================================
     # TaskFullTime components. Polymorphism and SimpleHistory don't play nice.
     # ==========================================================================
-    FIELDS_TASK_FULL_TIME = (
-        "is_full_time",
-    )
+    FIELDS_TASK_FULL_TIME = ("is_full_time",)
 
     is_full_time = BooleanField(
         default=False, verbose_name="Is full-time", help_text="If set, makes this task counts as being 1 FTE worth of load. Recursively defined."
@@ -380,7 +378,6 @@ def calculate_load(self, students: int | None, is_first_time: bool = False) -> f
             return load_calc * self.load_multiplier
 
 
-
 @receiver(pre_save, sender=Task)
 def update_task_name(sender: Type[Task], instance: Task, **kwargs):
     """
@@ -391,6 +388,7 @@ def update_task_name(sender: Type[Task], instance: Task, **kwargs):
     """
     instance.name = instance.get_name()
 
+
 #
 # @receiver(post_delete, sender=Task)
 # def update_related_models(sender: Type[Task], instance: Task, **kwargs):

Makefile

@@ -135,12 +135,8 @@ class StaffDetail(Page):
         editable=False,
         actions__submit=None,
     )
-    assignments_editable = AssignmentStaffEditTable(
-        include=lambda user, **_: user.is_staff
-    )
-    assignments = AssignmentStaffTable(
-        include=lambda user, **_: not user.is_staff
-    )
+    assignments_editable = AssignmentStaffEditTable(include=lambda user, **_: user.is_staff)
+    assignments = AssignmentStaffTable(include=lambda user, **_: not user.is_staff)
 
 
 class StaffList(Page):

physics_workload/app/models/task.py

@@ -61,9 +61,8 @@ class Meta:
 
 
 class AssignmentStaffEditTable(EditTable):
-    """
+    """ """
 
-    """
     class Meta:
         auto = dict(
             model=Assignment,

physics_workload/app/pages/staff/__init__.py

@@ -6,8 +6,9 @@
 import os
 import sys
 
+
 def main():
-    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'core.settings')
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "core.settings")
     try:
         from django.core.management import execute_from_command_line
     except ImportError as exc:
@@ -18,5 +19,6 @@ def main():
         ) from exc
     execute_from_command_line(sys.argv)
 
-if __name__ == '__main__':
+
+if __name__ == "__main__":
     main()

physics_workload/app/tables/assignment.py

@@ -140,7 +140,7 @@ extend-select = ["I"]
 [tool.ruff.lint.isort]
 combine-as-imports = true
 default-section = "third-party"
-known-first-party = ["physics_workload"]
+known-first-party = ["physics_workload", "app", "core", "users"]
 section-order = ["future", "standard-library", "third-party", "first-party", "local-folder"]
 
 [tool.ruff.lint.per-file-ignores]