Merge pull request #59 from jarrodmillman/ruff-conf

Update ruff config

Author: jarrodmillman

.git-blame-ignore-revs

@@ -0,0 +1 @@
+cdd3f5ae3b842e9e1ddfa76e9efa02c2eba337bd

.pre-commit-config.yaml

@@ -18,28 +18,19 @@ repos:
       - id: mixed-line-ending
       - id: trailing-whitespace
 
-  - repo: https://github.com/psf/black
-    rev: 3702ba224ecffbcec30af640c149f231d90aebdb # frozen: 24.4.2
-    hooks:
-      - id: black
-
-  - repo: https://github.com/adamchainz/blacken-docs
-    rev: 4c97c4a0d921007af6fefae92d8447cfbf63720b # frozen: 1.18.0
-    hooks:
-      - id: blacken-docs
-
   - repo: https://github.com/pre-commit/mirrors-prettier
     rev: ffb6a759a979008c0e6dff86e39f4745a2d9eac4 # frozen: v3.1.0
     hooks:
       - id: prettier
         files: \.(css|html|md|yml|yaml)
         args: [--prose-wrap=preserve]
 
-  - repo: https://github.com/asottile/pyupgrade
-    rev: 32151ac97cbfd7f9dcd22e49516fb32266db45b4 # frozen: v3.16.0
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: 1dc9eb131c2ea4816c708e4d85820d2cc8542683 # frozen: v0.5.0
     hooks:
-      - id: pyupgrade
-        args: [--py38-plus]
+      - id: ruff
+        args: ["--fix", "--show-fixes", "--exit-non-zero-on-fix"]
+      - id: ruff-format
 
 ci:
   autofix_prs: false

devstats/__main__.py

@@ -1,15 +1,13 @@
-import json
+import collections
 import os
 import re
 import sys
 from glob import glob
-import collections
 
 import click
-import requests
 
+from .publish import publish, template
 from .query import GithubGrabber
-from .publish import template, publish
 
 
 class OrderedGroup(click.Group):

devstats/publish.py

@@ -1,9 +1,8 @@
+import functools
 import os
-import sys
-from glob import glob
-import shutil
 import re
-import functools
+import shutil
+from glob import glob
 
 import click
 

devstats/query.py

@@ -1,8 +1,4 @@
 import json
-import os
-import re
-import sys
-from glob import glob
 
 import requests
 
@@ -98,7 +94,7 @@ def get_all_responses(query, query_type, headers):
     Helper function to bypass GitHub GraphQL API node limit.
     """
     # Get data from a single response
-    print(f"Retrieving first page...", end="", flush=True)
+    print("Retrieving first page...", end="", flush=True)
     initial_data = send_query(query, query_type, headers)
     data, last_cursor, total_count = parse_single_query(initial_data, query_type)
 

pyproject.toml

@@ -38,6 +38,43 @@ homepage = "https://github.com/scientific-python/devstats"
 [tool.changelist]
 ignored_user_logins = ["dependabot[bot]", "pre-commit-ci[bot]", "web-flow"]
 
+[tool.ruff.lint]
+extend-select = [
+  "B",        # flake8-bugbear
+  "I",        # isort
+  "ARG",      # flake8-unused-arguments
+  "C4",       # flake8-comprehensions
+  "EM",       # flake8-errmsg
+  "ICN",      # flake8-import-conventions
+  "G",        # flake8-logging-format
+  "PGH",      # pygrep-hooks
+  "PIE",      # flake8-pie
+  "PL",       # pylint
+  "PT",       # flake8-pytest-style
+#  "PTH",      # flake8-use-pathlib
+  "RET",      # flake8-return
+  "RUF",      # Ruff-specific
+  "SIM",      # flake8-simplify
+  "T20",      # flake8-print
+  "UP",       # pyupgrade
+  "YTT",      # flake8-2020
+  "EXE",      # flake8-executable
+  "NPY",      # NumPy specific rules
+  "PD",       # pandas-vet
+  "FURB",     # refurb
+  "PYI",      # flake8-pyi
+]
+ignore = [
+  "PLR09",    # Too many <...>
+  "PLR2004",  # Magic value used in comparison
+  "ISC001",   # Conflicts with formatter
+  "ARG002",   # Unused method argument
+  "EM101",    # Exception must not use a string literal
+  "SIM115",   # Use context handler for opening files
+  "RET504",   # Unnecessary assignment before `return` statement
+  "T201",     # `print` found
+]
+
 [tool.setuptools.packages.find]
 include = ["devstats*"]