Stop skipping E722, remove dead logic guards. Update contributing docs about testing

Author: timlnx

.github/workflows/python.yml

@@ -35,7 +35,7 @@ jobs:
 
       - name: Pre-Tests code smell validation
         run: |
-          pycodestyle -v --ignore=E501,E722 bitmath/__init__.py tests/
+          pycodestyle -v --ignore=E501 bitmath/__init__.py tests/
           flake8 --select=F bitmath/__init__.py tests/
 
       - name: Run Unit Tests

Makefile

@@ -207,7 +207,7 @@ ci-pycodestyle:
 	@echo "#############################################"
 	@echo "# Running PEP8 Compliance Tests in virtualenv"
 	@echo "#############################################"
-	. $(NAME)env3/bin/activate && pycodestyle -v --ignore=E501,E722 bitmath/__init__.py tests/*.py
+	. $(NAME)env3/bin/activate && pycodestyle -v --ignore=E501 bitmath/__init__.py tests/*.py
 
 ci-flake8:
 	@echo ""

bitmath/__init__.py

@@ -1663,10 +1663,7 @@ def parse_string(s: str | numbers.Number, system: int = NIST, strict: bool = Tru
             val = float(val)
         except ValueError:
             raise
-        try:
-            return unit_class(val)
-        except:  # pragma: no cover
-            raise ValueError(f"Can't parse string {s} into a bitmath object")
+        return unit_class(val)
 
     else:
         # strict=False path (formerly parse_string_unsafe)

docsite/source/contributing.rst

@@ -43,12 +43,13 @@ Two static analysis checks run on every pull request as part of the
 GitHub Actions CI workflow, and locally via ``make ci``:
 
 * ``pycodestyle`` — checks code style, with **E501** (line too long)
-  and **E722** (bare ``except``) ignored.
+  ignored.
 * ``flake8 --select=F`` — runs pyflakes error checks only (undefined
   names, unused imports, etc.). Style checks are disabled.
 
-A PR cannot be merged until both pass. Run ``make ci`` locally to
-check before submitting.
+A PR cannot be merged until both pass. If you want to save time you
+can run ``make ci`` locally to check before submitting and waiting on
+the GitHub runners to report back.
 
 
 .. _contributing_commit_messages:
@@ -102,7 +103,7 @@ This means the minimum supported version tracks the oldest Python still
 included in a supported EPEL target:
 
 * **RHEL 10 / EPEL 10** — Python 3.12
-* **RHEL 9 / EPEL 9** — Python 3.9, 3.11
+* **RHEL 9 / EPEL 9** — Python 3.9, 3.10, 3.11
 
 The CI matrix tests all versions in this range. When a RHEL major
 release reaches end-of-life and is dropped from EPEL, the corresponding