Skip to content

fix(rustfmt): add cargoManifestPath back#516

Merged
sandydoo merged 1 commit into
cachix:masterfrom
Pyxels:fix/rustfmt/cargoManifestPath
Oct 16, 2024
Merged

fix(rustfmt): add cargoManifestPath back#516
sandydoo merged 1 commit into
cachix:masterfrom
Pyxels:fix/rustfmt/cargoManifestPath

Conversation

@Pyxels
Copy link
Copy Markdown
Contributor

@Pyxels Pyxels commented Oct 15, 2024

It seems that #504 removed the application of the option of settings.rust.cargoManifestPath for rustfmt.

This PR readds that option so that setting settings.rust.cargoManifestPath (or the new rustfmt.settings.manifest-path) adds the --manifest-path argument to rustfmt.

hooks = {
  rustfmt.enable = true;
};
settings.rust.cargoManifestPath = "backend/Cargo.toml";
}

now becomes:

"entry": "/nix/store/04q1sh6cia6ar7zap273v3vpisf5mj7q-rustfmt-wrapped/bin/cargo-fmt fmt --all --manifest-path backend/Cargo.toml -- --color always"

@sandydoo sandydoo added the bug Something isn't working label Oct 15, 2024
@sandydoo sandydoo merged commit 307718f into cachix:master Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants