-
Notifications
You must be signed in to change notification settings - Fork 898
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[unstable option] unstable_features #3387
Comments
So to use There is something I miss |
According to this issue it behaves the way it is supposed to behave. However, I'd still argue that it should behave differently. Our usecase is to compile our code against stable Rust but we like to use unstable formatting features. Currently, this forces us install two versions of Rust in CI which
|
On 2.0, we will remove this and enable all unstable features on the nightly channel by default. On the stable, every unstable feature will be disabled. |
Just to clarify: I believe stable/nightly rust and stable/unstable formatting features are orthogonal to each other. |
This will need to be re-decided for 1.x |
I totally agree on this one. It's really annoying, and it's the thing across entire rust toolchain. Why do we need to explicitly install the nightly version of the toolchain to use an unstable feature? Why is it not enough to pass With |
For anyone else who lands here: YOU CAN USE UNSTABLE FEATURES ON STABLE TOOLCHAN, but you have to specify them via the CLI parameters like this: cargo fmt -- --config unstable_features=true --config error_on_unformatted=true For the reasoning why you can use them only via CLI and can't use them via a config file see #5511 (comment) |
I was confused about this and thought this meant that I could enable just the unstable_features via cli and that would cause the rustfmt.toml config file values to just work, but you have to enable every config option that you'd like to use. This seems like an unfortunate and non-intuitive way of making this work. |
Tracking issue for unstable_features
The text was updated successfully, but these errors were encountered: