Skip to content

Move to rust#12

Merged
timkpaine merged 11 commits intonbprint:mainfrom
Point72:tkp/rs
Feb 20, 2026
Merged

Move to rust#12
timkpaine merged 11 commits intonbprint:mainfrom
Point72:tkp/rs

Conversation

@timkpaine
Copy link
Member

No description provided.

@timkpaine timkpaine force-pushed the tkp/rs branch 2 times, most recently from 04ddf79 to d9744a3 Compare February 20, 2026 16:56
When running with pytest-cov, sys.gettrace() is set, which causes
is_under_debugger() to return True. This makes run_and_report() re-raise
exceptions instead of handling them gracefully with sys.exit(1).

By patching is_under_debugger to return False in the tests, we ensure
they work correctly regardless of whether coverage is being collected.
- Exclude lerna/tests from the wheel build to avoid ImportPathMismatchError
  when running pytest against the installed package
- Change cibuildwheel test-command to simple import test since full tests
  are run by 'make tests' separately
- Rust: normalize PathBuf outputs with .replace('\\', "/") in job/mod.rs and job_runner/mod.rs
- Python tests: quote and normalize sweep.dir overrides to handle Windows backslashes
- Fixes 26 Windows test failures from path separator mismatches
@timkpaine timkpaine marked this pull request as ready for review February 20, 2026 23:00
@timkpaine timkpaine merged commit 09245a1 into nbprint:main Feb 20, 2026
3 checks passed
@timkpaine timkpaine deleted the tkp/rs branch February 20, 2026 23:01
@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 219 files with indirect coverage changes

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

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