Skip to content

update pre-commit hooks and development tooling docs#11618

Open
xtqqczze wants to merge 2 commits intouutils:mainfrom
xtqqczze:pre-commit
Open

update pre-commit hooks and development tooling docs#11618
xtqqczze wants to merge 2 commits intouutils:mainfrom
xtqqczze:pre-commit

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented Apr 3, 2026

  • Remove the rust-clippy pre-commit hook as it is not fast enough to be usable and fails on macOS.
  • Update the development tooling docs as parts were outdated or unclear.

Justification

In my opinion, it's far more likely for a contributor to forget to update the fuzz lockfile than run clippy. There is more value if the hooks run fast and cover easily overlooked issues.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/cut/bounded-memory (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/follow-name (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/symlink (fails in this run but passes in the 'main' branch)

@xtqqczze

This comment was marked as resolved.

@sylvestre
Copy link
Copy Markdown
Contributor

sorry but i do value the clippy run

@oech3
Copy link
Copy Markdown
Contributor

oech3 commented Apr 3, 2026

We can skip this if --print host-tuple contains macos instead of removing.

@xtqqczze
Copy link
Copy Markdown
Contributor Author

xtqqczze commented Apr 3, 2026

I will try to progress #11016 in order fix clippy for macOS.

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