Nick Crews

Results 281 comments of Nick Crews

@cpcloud anything I can do to help move this forward/easier to review?

Thanks for the poke. I have a lot of other [PRs in flight](https://github.com/ibis-project/ibis/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+author%3ANickCrews) at the moment with ibis, many of which I think are a lot closer to the finish...

One thing that should get merged first is #9334, where I fix a datashape bug that is exposed in this PR. Once that is in, then this PR will be...

Once https://github.com/ibis-project/ibis/pull/9559 lands then I think the scope of this will be significantly reduced and this should be much easier, thank you for your reviews!

OK, this is much simpler now, there are no functionality changes, only API changes. The one question in the air is what to do with the old APIS. Can you...

I want the user experience to be consistent, and only one of the APIs to be surfaced in the docs and all the examples. I think I'd be fine not...

I'm still motivated to get this merged. I am trying to keep the original post at the top of this PR in sync with the changes, to provide a good...

@cpcloud gentle ping, whenever you get the chance :)

@cpcloud looks like the docs CI is failing just for permissions issues, so I think this is good to merge! (If you confirm then in the future I will be...

@cpcloud I added the `breaking change` label to this PR, since it does hard-break Value.cases(), as described in the OP at the top.