Clean up code, update workflows, fix ruff issues

Author: LilSpazJoekp

.coveragerc

@@ -0,0 +1,5 @@
+[report]
+exclude_lines =
+    @abstract
+    if TYPE_CHECKING:
+    pragma: no cover

.github/workflows/codeql.yml

@@ -6,11 +6,12 @@ jobs:
       id-token: write
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Set up Python
-        uses: actions/setup-python@v2
+        uses: actions/setup-python@v5
         with:
-          python-version: '3.x'
+          cache: pip
+          python-version: 3.x
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip

.github/workflows/pypi.yml

@@ -1,6 +1,6 @@
 jobs:
   analysis:
-    name: Scorecards analysis
+    name: Scorecard analysis
     permissions:
       id-token: write
       security-events: write
@@ -23,14 +23,14 @@ jobs:
           path: results.sarif
           retention-days: 5
       - name: Upload to code-scanning
-        uses: github/codeql-action/upload-sarif@1b1aada464948af03b950897e5eb522f92603cc2 # v3.24.9
+        uses: github/codeql-action/upload-sarif@v3
         with:
           sarif_file: results.sarif
-name: Scorecards supply-chain security
+name: Scorecard supply-chain security
 on:
   branch_protection_rule:
   push:
     branches: [ master ]
   schedule:
-    - cron: 30 1 * * 6
+    - cron: 0 15 * * 1
 permissions: read-all

.github/workflows/scorecards-analysis.yml

@@ -58,7 +58,7 @@ def authorize(self, code: str) -> str | None:
         return authorizer.refresh_token
 
     @_deprecate_args("access_token", "expires_in", "scope")
-    def implicit(self, *, access_token: str, expires_in: int, scope: str) -> None:
+    def implicit(self, *, access_token: str, expires_in: int, scope: str):
         """Set the active authorization to be an implicit authorization.
 
         :param access_token: The access_token obtained from Reddit's callback.

praw/models/auth.py

@@ -21,7 +21,7 @@ def __len__(self) -> int:
         """Return the number of items in the Listing."""
         return len(getattr(self, self.CHILD_ATTRIBUTE))
 
-    def __setattr__(self, attribute: str, value: Any) -> None:
+    def __setattr__(self, attribute: str, value: Any):
         """Objectify the ``CHILD_ATTRIBUTE`` attribute."""
         if attribute == self.CHILD_ATTRIBUTE:
             value = self._reddit._objector.objectify(value)

praw/models/listing/listing.py

@@ -518,7 +518,7 @@ def __len__(self) -> int:
         """
         return len(self.link_ids)
 
-    def __setattr__(self, attribute: str, value: Any) -> None:
+    def __setattr__(self, attribute: str, value: Any):
         """Objectify author, subreddit, and sorted_links attributes."""
         if attribute == "author_name":
             self.author = self._reddit.redditor(value)

praw/models/reddit/collections.py

@@ -86,7 +86,7 @@ def user_selection(self) -> PollOption | None:
             return None
         return self.option(self._user_selection)
 
-    def __setattr__(self, attribute: str, value: Any) -> None:
+    def __setattr__(self, attribute: str, value: Any):
         """Objectify the options attribute, and save user_selection."""
         if attribute == "options" and isinstance(value, list):
             value = [PollOption(self._reddit, option) for option in value]

praw/models/reddit/poll.py

@@ -95,7 +95,7 @@ def read_only(self) -> bool:
         return self._core == self._read_only_core
 
     @read_only.setter
-    def read_only(self, value: bool) -> None:
+    def read_only(self, value: bool):
         """Set or unset the use of the ReadOnlyAuthorizer.
 
         :raises: :class:`.ClientException` when attempting to unset ``read_only`` and
@@ -441,6 +441,7 @@ def request(self, *args, **kwargs):
     def _check_for_async(self):
         if self.config.check_for_async:  # pragma: no cover
             try:
+                # noinspection PyUnresolvedReferences
                 shell = get_ipython().__class__.__name__
                 if shell == "ZMQInteractiveShell":
                     return

praw/reddit.py

@@ -85,7 +85,7 @@ class FileTokenManager(BaseTokenManager):
 
     """
 
-    def __init__(self, filename: str) -> None:
+    def __init__(self, filename: str):
         """Initialize a :class:`.FileTokenManager` instance.
 
         :param filename: The file the contains the refresh token.