Skip to content

Update to cel-interpreter 0.10#3

Merged
hardbyte merged 5 commits intomainfrom
update/cel-interpreter-0.10
Aug 1, 2025
Merged

Update to cel-interpreter 0.10#3
hardbyte merged 5 commits intomainfrom
update/cel-interpreter-0.10

Conversation

@hardbyte
Copy link
Owner

No description provided.

@hardbyte hardbyte requested a review from Copilot August 1, 2025 00:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the CEL Python bindings to use cel-interpreter 0.10.0, bringing significant improvements to mixed-type arithmetic, error handling, and overall robustness. The update includes automatic type coercion for better int/float compatibility, enhanced error handling with proper Python exceptions, and comprehensive test coverage improvements.

Key changes:

  • Updated cel-interpreter from 0.9.0 to 0.10.0 with intelligent type coercion for mixed arithmetic
  • Enhanced error handling with panic protection and proper Python exception mapping
  • Comprehensive test suite expansion with 10+ new test modules covering edge cases and functionality

Reviewed Changes

Copilot reviewed 27 out of 60 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/lib.rs Core Rust implementation with PyO3 0.25.0 migration, type coercion, and error handling
src/context.rs Updated context management with new PyO3 APIs and enhanced documentation
tests/test_*.py Comprehensive test suite with 10 new test modules covering types, datetime, CLI, etc.
python/cel/cli.py Full-featured CLI with Rich formatting, REPL, and syntax highlighting
pyproject.toml Build system updates with new dependencies and tooling configuration
Cargo.toml Dependency updates to cel-interpreter 0.10.0 and PyO3 0.25.0

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@hardbyte hardbyte merged commit c7d7560 into main Aug 1, 2025
18 checks passed
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.

2 participants