Skip to content

Conversation

robem
Copy link
Contributor

@robem robem commented Sep 16, 2025

Description

Add cargo machete to lefthook. This is an optimistic PR as I'm not fully aware of the pipeline environment and whether cargo-machete is present as it is not a rustup component.

Motivated by #554 (comment)

Checklist

Check the boxes that apply (put an x in the brackets, like [x]). You can also check boxes after the PR is created.

❤️ Thank you for your contribution!

@coveralls
Copy link

Pull Request Test Coverage Report for Build 17778467557

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 27.748%

Totals Coverage Status
Change from base Build 17776886912: 0.0%
Covered Lines: 1156
Relevant Lines: 4166

💛 - Coveralls

@robem
Copy link
Contributor Author

robem commented Sep 16, 2025

If cargo-machete cannot be assumed then command -v cargo-machete >/dev/null 2>&1 && cargo machete for a less disruptive change if desired.

@Xenira
Copy link
Collaborator

Xenira commented Sep 30, 2025

@robem sorry for the delay. I would not assume machete to be present.

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