Skip to content

chore: fix noxfile, CI pass gate, and stale config#712

Merged
henryiii merged 2 commits into
mainfrom
chore/infra-fixes
Jun 15, 2026
Merged

chore: fix noxfile, CI pass gate, and stale config#712
henryiii merged 2 commits into
mainfrom
chore/infra-fixes

Conversation

@henryiii

Copy link
Copy Markdown
Member

🤖 AI text below 🤖

Summary

  • noxfile docs session: session.install("-e.", doc_deps) was passing the deps list as a single argument (TypeError at runtime). Fixed to session.install("-e.", *doc_deps).
  • noxfile free-threaded env: Was inspecting sys.version_info/sysconfig of the nox host interpreter and mutating os.environ["PYTHON_GIL"], leaking into subsequent sessions. Now detects free-threaded builds from the "t" suffix on session.python and passes env={"PYTHON_GIL": "0"} to session.run(...) only. Applied to both lite and tests sessions (CI runs lite on 3.14t). Drops unused os/sys/sysconfig imports.
  • CI pass gate: Added if: always() and a step that fails when any upstream job is in failure or cancelled state, so skipped jobs no longer satisfy branch protection.
  • numba lower bound: Aligned test-optional group from >=0.57 to >=0.62 to match the published [numba] extra.
  • Duplicate pylint disable: Removed the second "cast_python_value" entry from pyproject.toml.
  • Dead requirements-txt-fixer hook: Removed from .pre-commit-config.yaml (no requirements*.txt files exist in the repo).
  • blacken-docs black pin: Bumped additional_dependencies: [black~=24.0]black~=25.0; pre-commit.ci autoupdate never touches additional_dependencies.

Part of #711

All changes validated: yaml.safe_load on CI YAML OK, uvx nox -l lists sessions without error, prek -a --quiet passes clean.

🤖 Generated with Claude Code

- noxfile: unpack doc_deps in docs session (was passing list, causing TypeError)
- noxfile: detect free-threaded Python from session.python suffix ('t') instead
  of inspecting nox-host sysconfig; pass PYTHON_GIL=0 via env= kwarg rather
  than mutating os.environ; apply to both lite and tests sessions
- noxfile: drop unused os/sys/sysconfig imports
- ci.yml: add if: always() to pass job and fail-step so skipped upstream jobs
  don't bypass branch protection
- pyproject.toml: align test-optional numba lower bound to >=0.62 (matches extra)
- pyproject.toml: remove duplicate cast_python_value in pylint disable list
- .pre-commit-config.yaml: remove requirements-txt-fixer (no requirements*.txt)
- .pre-commit-config.yaml: bump blacken-docs black pin from ~=24.0 to ~=25.0

Assisted-by: ClaudeCode:claude-sonnet-4-6
@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.09%. Comparing base (3e32955) to head (21bbb98).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #712   +/-   ##
=======================================
  Coverage   87.09%   87.09%           
=======================================
  Files          96       96           
  Lines       11231    11231           
=======================================
  Hits         9782     9782           
  Misses       1449     1449           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .pre-commit-config.yaml Outdated
@henryiii henryiii marked this pull request as ready for review June 11, 2026 23:12
@henryiii henryiii merged commit 2f5b899 into main Jun 15, 2026
19 checks passed
@henryiii henryiii deleted the chore/infra-fixes branch June 15, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant