feat: warnings.warn instead of print

this does not cover all cases, but it covers some

Author: faretek1

scratchattach/site/backpack_asset.py

@@ -3,6 +3,7 @@
 import json
 import time
 import logging
+import warnings
 
 from ._base import BaseSiteComponent
 from scratchattach.utils import exceptions
@@ -39,7 +40,7 @@ def __init__(self, **entries):
         self.__dict__.update(entries)
 
     def update(self):
-        print("Warning: BackpackAsset objects can't be updated")
+        warnings.warn("Warning: BackpackAsset objects can't be updated")
         return False  # Objects of this type cannot be updated
 
 

scratchattach/site/comment.py

@@ -1,6 +1,7 @@
 """Comment class"""
 from __future__ import annotations
 
+import warnings
 from typing import Union, Optional, Any
 from typing_extensions import assert_never  # importing from typing caused me errors
 from enum import Enum, auto
@@ -48,7 +49,7 @@ def __init__(self, **entries):
             self.source = "Unknown"
 
     def update(self):
-        print("Warning: Comment objects can't be updated")
+        warnings.warn("Warning: Comment objects can't be updated")
         return False  # Objects of this type cannot be updated
 
     def _update_from_dict(self, data):

scratchattach/site/forum.py

@@ -1,6 +1,7 @@
 """ForumTopic and ForumPost classes"""
 from __future__ import annotations
 
+import warnings
 from dataclasses import dataclass, field
 from typing import Optional, Any
 from urllib.parse import urlparse, parse_qs
@@ -114,7 +115,7 @@ def posts(self, *, page=1, order="oldest") -> list[ForumPost]:
             list<scratchattach.forum.ForumPost>: A list containing the posts from the specified page of the forum topic 
         """
         if order != "oldest":
-            print("Warning: All post orders except for 'oldest' are deprecated and no longer work") # For backwards compatibility
+            warnings.warn("Warning: All post orders except for 'oldest' are deprecated and no longer work") # For backwards compatibility
 
         posts = []
 
@@ -143,7 +144,7 @@ def posts(self, *, page=1, order="oldest") -> list[ForumPost]:
                     link = breadcrumb_ul.find_all('a')[1]  # Get the right anchor tag
                     topic_category = link.text.strip()  # Extract and strip text content
             except Exception as e:
-                print(f"Warning: Couldn't scrape topic category for topic {self.id} - {e}")
+                warnings.warn(f"Warning: Couldn't scrape topic category for topic {self.id} - {e}")
                 topic_category = ""
 
             # get corresponding posts: