Skip to content

Conversation

leoleoasd
Copy link

  • Change language to rust to let pre-commit bootstrap rust
  • Add an empty cargo.toml to fool the build system

Currently, this would fail on pre-commit.ci since it doesn't declare itself that it requires cargo.

See: before, after.

cargo fmt will work now. cargo clippy still won't work since it requires access to the internet.

Terseus added a commit to Terseus/yet-another-md5 that referenced this pull request Nov 4, 2023
Unfortunately the hooks in pre-commit-rust doesn't work on pre-commit-ci
because it have language as "system" instead of "rust".

See doublify/pre-commit-rust#28

Although we may get away of this using "repo: local" I don't think is
worth the effort.
@frank1010111
Copy link

this would be really nice

@eirnym
Copy link

eirnym commented May 3, 2024

I prefer this as an option as exact version of rust is important. E.g. some projects use nightly starting from specific version and cargo fmt would not recognize formatting options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants