Alexander Huszagh

Results 294 comments of Alexander Huszagh

This still seems to depend on a nightly compiler, as documented in [doctest-xcompile](https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#doctest-xcompile) - https://github.com/rust-lang/cargo/issues/7040 - https://github.com/rust-lang/rust/issues/64245 If the subcommand is `test` and the compiler is nightly, we may be...

This still needs to fix deployment and CI: we need to make sure the base image is always built first, and that it is staged for our other image build....

This is reproducing, but has some interesting quirks. I'll be commenting here and trying to debug. Current information: ``` $ docker run --rm -it --entrypoint bash \ -e CROSS_DOCKER_IN_DOCKER=true \...

rust-docker currently hasn't been maintained in over 7 years, and we support much more than just Docker now (including podman, and I've gotten Lima to work as well), and this...

This should also have the verbosity settings: we should also parse `quiet`. This should also be passed to `rustup`.

Is this closes now that we can erase lines via #911? We should have essentially full support of the functionality of cargo's diagnostics itself.

This is still replicating as of the latest commit today, with the same error. We might need to update our include directories, I'll try to debug this.

It might also be worth linking this with #797, and adding the diagnostics using `msg_info.note` or something similar. Something like: ```diff diff --git a/src/docker/mod.rs b/src/docker/mod.rs index cd80b20..6fc78c4 100644 --- a/src/docker/mod.rs...

Removed some of my comments since the rationale for the PR was made clearer elsewhere: the goal is so every external call to a `rustup`, `cargo`, etc. command is only...

Needs a changelog entry but other than that, lgtm