-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Changes the default verbosityRestore to be a more default value #12298
Conversation
Thanks @aarondandy for the PR. I do not know of any reason why it should be detailed verbosity, and hence this looks good to me.
|
I fixed the versions and duplicate files up as you requested. Docs: https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-restore#options |
@aarondandy this PR needs to be synced base branch. Please do so and I will then merge it. |
@bishal-pdMSFT Can you double check that the patch increment is still correct? I am not familiar with how you version things in these PRs. |
I'm going to stop updating this PR to be up to date with master. Let me know before hand if you want to attempt merging this again and I can fix it up then. If you think this is not likely to be merged I can just close it. |
This PR is stale because it has been open for 180 days with no activity. Remove the stale label or comment on the PR otherwise this will be closed in 5 days |
This proposed change reduces the level of noise in the output from
dotnet restore
as outlined in #12297 . I am unable to run the build or tests on my machine so I hope this does not conflict with any possible regression tests 🤞