Merge pull request #2 from kjmeagher/pre-commit-ci-update-config

autoupdate pre-commit hooks
kjmeagher · Apr 4, 2024 · d02a427 · d02a427
2 parents f4fde25 + d91dc56
commit d02a427
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 11 deletions.
diff --git a/.github/workflows/unit_tests.yml b/.github/workflows/unit_tests.yml
@@ -3,7 +3,11 @@
 # SPDX-License-Identifier: BSD-2-Clause
 
 name: Tests
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   Tests:
     runs-on: ${{ matrix.os }}

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -10,24 +10,21 @@ repos:
     rev: v4.0.0-alpha.8
     hooks:
       - id: prettier
-  - repo: https://github.com/psf/black
-    rev: 23.12.1
-    hooks:
-      - id: black
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.9
+    rev: v0.3.5
     hooks:
       - id: ruff
-        args: [--fix]
+        args: [--fix, --show-fixes]
+      - id: ruff-format
   - repo: https://github.com/pycqa/pylint
-    rev: v3.0.3
+    rev: v3.1.0
     hooks:
       - id: pylint
         files: ehist
         args: [--ignore=E401]
         additional_dependencies: [numpy, matplotlib]
   - repo: https://github.com/fsfe/reuse-tool
-    rev: v2.1.0
+    rev: v3.0.1
     hooks:
       - id: reuse
   - repo: https://github.com/codespell-project/codespell
@@ -37,7 +34,7 @@ repos:
         additional_dependencies:
           - tomli
   - repo: https://github.com/Lucas-C/pre-commit-hooks
-    rev: v1.5.4
+    rev: v1.5.5
     hooks:
       - id: forbid-crlf
       - id: forbid-tabs

diff --git a/ehist/bayesian_blocks.py b/ehist/bayesian_blocks.py
@@ -41,6 +41,7 @@
 .. [1] https://ui.adsabs.harvard.edu/abs/2013ApJ...764..167S
 .. [2] https://www.astroml.org/ https://github.com//astroML/astroML/
 """
+
 import warnings
 from inspect import signature
 

diff --git a/tests/test_hist1d.py b/tests/test_hist1d.py
@@ -2,6 +2,8 @@
 #
 # SPDX-License-Identifier: BSD-2-Clause
 
+import functools
+import operator
 import unittest
 
 import numpy as np
@@ -20,7 +22,7 @@ def quadratic(x, a, b, c):
 
 class TestAxis(unittest.TestCase):
     def test_integer(self):
-        v = sum([i * [i] for i in range(10)], [])
+        v = functools.reduce(operator.iadd, [i * [i] for i in range(10)], [])
         h = Hist1D(v)
         assert_allclose(h.N, range(1, 10))
         assert_allclose(h.H, range(1, 10))