search

clarkmcc / cel-rust

Common Expression Language interpreter written in Rust
https://crates.io/crates/cel-interpreter
MIT License
362 stars 18 forks source link

Fix method/fn parsing and lookup #59

Closed alexsnaps closed 2 months ago

alexsnaps commented 2 months ago

Fix #53

I think this is the AST we discussed, let me know! Side note: had to change the test that tested for illegal compares, as now size looks up a variable only. Is this correct? or should an Ident be usable to explicitly reference a function?

alexsnaps commented 2 months ago

@clarkmcc You think you might be able to cut a new release... soonish?

clarkmcc commented 2 months ago

@alexsnaps I've got one more PR I'm hoping to review early next week. Is this an option for you in the short term?

[dependencies]
cel-interpreter = { git = "https://github.com/clarkmcc/cel-rust", branch = "master" }
alexsnaps commented 2 months ago

Sure thing! That's "soonish" enough! Thanks!