Skip to content
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

fix(rustfmt): add cargoManifestPath back #516

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

Pyxels
Copy link
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
4 checks passed
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