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

Cherry pick #5528 for verified release 2.0. #5529

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

sini
Copy link
Contributor

@sini sini commented Sep 2, 2021

See #5528 for details.

…am changes (#5528)

Pre-commit is using a version of dotnet and dotnet-format and has picked up a newer version of the utility that is not compatible with .NET 3.1 or 5.x -- it installs it's packages using dotnet tool install. This version's default entrypoint is also incompatible with our project structure, as the style and analyzer subcommands are now run by default and do not support the --folder option.

The entrypoint can be over-ridden to specify the whitespace module, which should match previous behavior.
@sini sini changed the base branch from main to 2.0-verified September 2, 2021 22:20
@sini sini requested a review from dongruoping September 2, 2021 22:20
@dongruoping dongruoping merged commit 71f488f into 2.0-verified Sep 2, 2021
@delete-merged-branch delete-merged-branch bot deleted the 2.0-verified-precommit-fix branch September 2, 2021 23:02
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants