rust-lang / rust-mode

Emacs configuration for Rust
Apache License 2.0
1.09k stars 176 forks source link

Skip test if Cargo isn’t installed. #459

Closed phst closed 1 year ago

phst commented 1 year ago

This test requires Cargo and fails if it’s not installed.

brotzeit commented 1 year ago

So the test doesn't run. Any idea why ?

phst commented 1 year ago

rust-buffer-project invokes Cargo, cf. https://github.com/rust-lang/rust-mode/blob/2d6bc53f526623e585aae5855a81a13b709df97d/rust-cargo.el#L45. If that fails, it signals an error.

brotzeit commented 1 year ago

Sorry, wasn't clear enough ;)

I mean the ci doesn't seem to run the test, otherwise it would fail because we don't install cargo.

phst commented 1 year ago

The GitHub runners have Cargo installed by default (https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools)

brotzeit commented 1 year ago

Interesting, thanks!