Skip to content

Comments

feat: add edge case tests, GitHub Actions CI, and ruff linting#64

Merged
maskedsyntax merged 3 commits intomainfrom
feat/testing-ci-linting
Feb 19, 2026
Merged

feat: add edge case tests, GitHub Actions CI, and ruff linting#64
maskedsyntax merged 3 commits intomainfrom
feat/testing-ci-linting

Conversation

@maskedsyntax
Copy link
Member

Summary

  • Added 37 new edge case tests covering empty DataFrames, constant/degenerate columns, infinite values, correlation edge cases, leakage without target, drift with disjoint/single-value columns, check filtering, and distribution boundary conditions
  • Created GitHub Actions CI workflow with pytest on Python 3.10-3.12 and ruff linting
  • Added ruff configuration to pyproject.toml with sensible rule selection (E, W, F, I, UP, B, SIM)
  • Fixed all lint issues: unused imports, explicit re-exports in __init__.py files, unused variables
  • Applied consistent ruff formatting across the entire codebase (54 files)

Test plan

  • All 119 tests pass (82 existing + 37 new edge case tests)
  • ruff check . passes clean
  • ruff format --check . passes clean
  • CI workflow validated locally

- Add 37 edge case tests covering empty DataFrames, constant columns,
  infinite values, correlation edge cases, leakage, drift, and more
- Add GitHub Actions CI workflow (pytest on Python 3.10-3.12, ruff lint)
- Add ruff configuration with sensible defaults for the project
- Add ruff as dev dependency
- Fix all ruff lint issues (unused imports, re-exports, unused vars)
- Apply ruff formatting across the entire codebase
@vercel
Copy link

vercel bot commented Feb 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hashprep Ready Ready Preview, Comment Feb 19, 2026 0:05am

@maskedsyntax maskedsyntax merged commit 47b45d5 into main Feb 19, 2026
6 checks passed
@maskedsyntax maskedsyntax deleted the feat/testing-ci-linting branch February 19, 2026 12:15
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