Skip to content

chore: CI + tests hardening#1

Open
AITactician wants to merge 2 commits into
mainfrom
chore/ci-and-tests
Open

chore: CI + tests hardening#1
AITactician wants to merge 2 commits into
mainfrom
chore/ci-and-tests

Conversation

@AITactician

Copy link
Copy Markdown
Owner

This PR improves CI consistency and hardens our local/CI testing experience.

Key changes

  • Update GitHub Actions workflow matrix to Python 3.9–3.12 and add robust validation steps
  • Add .env.ci with safe dummy values for CI-only usage
  • Add tooling configs (Black, isort) centralized in pyproject.toml for single source of truth
  • Refresh pre-commit hooks: JSON/YAML checks, JSON Schema validation, unit test hook
  • Regenerate requirements and dev lockfiles for reproducible installs
  • Add tests: risk/execution safety smoke tests and conftest for shared fixtures
  • Improve examples and scripts to align with loader usage

Impact

  • Faster, more reliable CI with better linting/formatting parity
  • Safer config validation and reproducible environments

Changelog

  • ci: refine matrix and add schema/config validations
  • build: add pyproject tool config; lock requirements
  • test: add safety smoke and risk/execution tests; conftest
  • docs: improve README/tests documentation alignment

Fixed issues

  • Closes #123

Notes

  • Local act run is currently blocked by Docker daemon on Apple silicon; CI will validate on ubuntu-latest runners.

- Update CI workflow matrix and validation steps
- Add .env.ci with safe dummy values for CI
- Add tooling configs (black, isort) in pyproject.toml
- Refresh pre-commit config with JSON and schema checks, unit tests hook
- Update requirements and dev requirements lockfiles
- Improve tests and examples for config loader

Closes: #123
- Add comprehensive .gitignore and ignore .ci-venv
- Remove committed .ci-venv from repo
- Apply isort to fix import ordering per CI
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