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

gracefully exit if we can't solve an issue with peer dependencies #11000

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

brettfo
Copy link
Contributor

@brettfo brettfo commented Nov 22, 2024

When attempting to perform an update, we currently can't do it if we detect a difference in the peer dependencies. Eventually this is a situation that should be solvable, but since it's currently not, it's better to gracefully exit the updater instead of throwing an error.

This is currently one of the biggest error traces in the telemetry.

@brettfo brettfo added the L: dotnet:nuget NuGet packages via nuget or dotnet label Nov 22, 2024
@brettfo brettfo requested a review from a team as a code owner November 22, 2024 17:58
@brettfo brettfo force-pushed the dev/brettfo/nuget-peer-dependencies branch from 2383854 to 6df0c9b Compare November 22, 2024 18:00
@brettfo brettfo force-pushed the dev/brettfo/nuget-peer-dependencies branch 2 times, most recently from 0476d96 to 32bfa70 Compare November 22, 2024 19:19
@randhircs randhircs force-pushed the dev/brettfo/nuget-peer-dependencies branch from 32bfa70 to 1b870a1 Compare November 22, 2024 23:40
@randhircs randhircs merged commit 50b93dc into main Nov 23, 2024
70 checks passed
@randhircs randhircs deleted the dev/brettfo/nuget-peer-dependencies branch November 23, 2024 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants