blark icon indicating copy to clipboard operation
blark copied to clipboard

apischema as a non-optional requirement?

Open klauer opened this issue 2 years ago • 1 comments

I'm considering making apischema a non-optional requirement. For those that are unaware, it's what allows blark to easily (de)serialize parsed code to/from JSON directly from the blark.transform dataclasses. It's currently made optional through a bit of hackery: https://github.com/klauer/blark/blob/abce326714cb9d4a7e0a2d6ad5d1cf8dc2c1110b/blark/transform.py#L26-L36

It's a reasonably light dependency, and it'd make things more convenient were it just a hard requirement.

Any thoughts, objections, emojis?

klauer avatar Oct 25 '23 16:10 klauer

Sounds perfectly reasonable to me!

engineerjoe440 avatar Oct 26 '23 00:10 engineerjoe440