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

Change the git pre-commit actions to *actually* use stable #6754

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

BenWiederhake
Copy link
Collaborator

This is in response to #6715: e8bb8df

The pre-commit actions have a higher MSRV than the overall project (1.70.0), so if 1.70.0 is the configured default, this lead to problems: #6730

This PR fixes the situation by specifying 'stable', whatever that means on the user's system.

Fixes #6730.

This is in response to uutils#6715: e8bb8df
The pre-commit actions have a higher MSRV than the overall project (1.70.0),
so if 1.70.0 is the configured default, this lead to problems (uutils#6730).
This commit fixes the situation by specifying 'stable', whatever that means on the user's system.
Copy link

GNU testsuite comparison:

GNU test failed: tests/timeout/timeout. tests/timeout/timeout is passing on 'main'. Maybe you have to rebase?

@sylvestre sylvestre merged commit 1f0b519 into uutils:main Sep 30, 2024
67 of 68 checks passed
@BenWiederhake BenWiederhake deleted the dev-stable-precommit branch September 30, 2024 22:24
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.

pre-commit: assumes specific rust version
2 participants