search

Chia-Network / clvm_tools

Tools for clvm development
Apache License 2.0
51 stars 32 forks source link

fix compatibility with most recent version of clvm_rs #100

Closed arvidn closed 1 year ago

arvidn commented 1 year ago

this PR renamed the run_chia_program() function to run_serialized_chia_program().

richardkiss commented 1 year ago

Shouldn't you bump the version number in setup.py too?

richardkiss commented 1 year ago

Interesting, it isn't in there. We should probably add it as an optional install. Maybe something like

    extras_require=dict(dev=dev_dependencies, rust=rust_dependencies),

here https://github.com/Chia-Network/clvm_tools/blob/main/setup.py#L48

In any case, I'll approve this.

richardkiss commented 1 year ago

And we should probably pin version numbers a bit tighter in python, like we do in rust.