Maximilian Roos

Results 1453 comments of Maximilian Roos

I agree this would be nice! previously we didn't want to add a yaml dependency to the basic `insta`. but: - possibly that's less of an issue now that we've...

This is blocked on MSRV. Cargo will have a new MSRV resolver soon, which will mean we won't have these PRs hanging around — they'll be open when they're ready....

@dependabot ignore this minor version

Can we adjust the example so `get_magic_name` is defined?

> But `get_magic_name` is not defined in the PRQL script because it is a user-defined function (UDF) that has been created on the database server using the `CREATE FUNCTION` SQL...

OK great. I think this can be handled by an s-string at the moment. If there were substantial demand we could add a `call_udf` function, but it might be so...

I definitely see the case here. I'm not sure the proposal quite fits — it uses a namespace to indicate a type. TBC, it's not crazy — maybe we store...

interesting q! what's the format of the other snapshot files? we could do a check of remaining files' headers and check they're insta files?

> Or add a `--test-runner-options`? I considered this. but then would `--test-runner-options="--status-level fail -- --nocapture"` give the same result as `cargo insta test -- --status-level fail -- --nocapture` would in...

on reflection, this code is overly complicated and would be much simpler if we didn't run doctests separately. so I got around to doing https://github.com/mitsuhiko/insta/pull/803, and we can merge a...