chore: improve pyproject.toml from sp-repo-review checks (#1511)

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

Author: henryiii

.pre-commit-config.yaml

@@ -26,16 +26,9 @@ repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.13.3
     hooks:
-      - id: ruff
+      - id: ruff-check
         args: [--fix, --show-fixes]
 
-
-  - repo: https://github.com/asottile/pyupgrade
-    rev: v3.20.0
-    hooks:
-      - id: pyupgrade
-        args: [--py38-plus]
-
   - repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
     rev: v2.15.0
     hooks:

pyproject.toml

@@ -45,9 +45,6 @@ test-pyodide = [
   {include-group = "test-core"}
 ]
 
-[lint.mccabe]
-max-complexity = 100
-
 [project]
 authors = [
   {name = "Jim Pivarski", email = "jpivarski@gmail.com"}
@@ -57,7 +54,6 @@ classifiers = [
   "Intended Audience :: Developers",
   "Intended Audience :: Information Technology",
   "Intended Audience :: Science/Research",
-  "License :: OSI Approved :: BSD License",
   "Operating System :: MacOS",
   "Operating System :: POSIX",
   "Operating System :: Unix",
@@ -114,12 +110,8 @@ Homepage = "https://github.com/scikit-hep/uproot5"
 version-file = "src/uproot/version.py"
 
 [tool.hatch.version]
-path = "src/uproot/__init__.py"
 source = "vcs"
 
-[tool.isort]
-profile = "black"
-
 [tool.pytest.ini_options]
 addopts = ["-ra", "--showlocals", "--strict-markers", "--strict-config"]
 filterwarnings = [
@@ -128,7 +120,7 @@ filterwarnings = [
   "ignore:unclosed transport <asyncio.sslproto._SSLProtocolTransport",  # https://github.com/aio-libs/aiohttp/issues/1115
   "ignore: .*will default to writing RNTuples.*:FutureWarning"
 ]
-log_cli_level = "info"
+log_cli_level = "INFO"
 markers = [
   "slow",
   "network",
@@ -140,39 +132,22 @@ testpaths = ["tests"]
 timeout = 600
 xfail_strict = true
 
+[tool.repo-review]
+ignore = ["PY004"]
+
 [tool.ruff]
 exclude = [
   "tests/*.py",
   "tests-cuda/*.py",
   "src/uproot/__init__.py",
   "docs-sphinx/*.py"
 ]
-src = ["src"]
 
 [tool.ruff.lint]
-ignore = [
-  "E501",
-  "E722",
-  "PLR",
-  "PLW0120",  # else on loop without break
-  "SIM118",  # key in dict, broken since uproot doesn't behave like a dict
-  "PGH003",  # too-broad type ignore
-  "SIM114",  # combine `if` branches using logical `or` operator
-  "S307",  # no eval allowed
-  "PLC1901",  # empty string is falsey (but I don't want to rely on such weak typing)
-  "RUF012",  # enforces type annotations on a codebase that lacks type annotations
-  "SIM103",  # interferes with my if-elif-elif-else blocks
-  "PLC0415",  # imports outside top level
-  "PLW1641"  # __eq__ without __hash__
-]
-select = [
-  "E",
-  "F",
-  "W",  # flake8
-  "B",
-  "B904",  # flake8-bugbear
+extend-select = [
+  "B",  # flake8-bugbear
   "I",  # isort
-  # "ARG",         # flake8-unused-arguments
+  # "ARG", # flake8-unused-arguments
   "C4",  # flake8-comprehensions
   "ISC",  # flake8-implicit-str-concat
   "PGH",  # pygrep-hooks
@@ -185,6 +160,19 @@ select = [
   "UP",  # pyupgrade
   "YTT"  # flake8-2020
 ]
+ignore = [
+  "PGH003",  # too-broad type ignore
+  "PLC0415",  # imports outside top level
+  "PLC1901",  # empty string is falsey (but I don't want to rely on such weak typing)
+  "PLR",
+  "PLW0120",  # else on loop without break
+  "PLW1641",  # __eq__ without __hash__
+  "RUF012",  # enforces type annotations on a codebase that lacks type annotations
+  "S307",  # no eval allowed
+  "SIM103",  # interferes with my if-elif-elif-else blocks
+  "SIM114",  # combine `if` branches using logical `or` operator
+  "SIM118"  # key in dict, broken since uproot doesn't behave like a dict
+]
 isort.required-imports = ["from __future__ import annotations"]
 
 [tool.ruff.lint.per-file-ignores]