search

caketop / python-starlark-go

🐍 Python bindings for starlark-go 🐍
https://python-starlark-go.readthedocs.io/
Apache License 2.0
20 stars 7 forks source link

Add context to EvalError #163

Closed jordemort closed 1 year ago

jordemort commented 1 year ago

Closes #142

colindean commented 1 year ago

Awesome!

jordemort commented 1 year ago

@colindean This was technically an API break because it changes the constructor of EvalError but I'm not sure it's worth 2.0 on its own

I was going to try and knock out these over the next couple weeks before I tag a new major version:

If you want this change in a published release sooner LMK and I'll tag a point release for you; I think it's OK to bend the semver rules since I doubt anyone is using the EvalError constructor directly.