Skip to content

better diagnostics for networking errors #6330

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

Merged
merged 3 commits into from
Mar 27, 2023
Merged

Conversation

tomerd
Copy link
Contributor

@tomerd tomerd commented Mar 25, 2023

motivation: improved user experience

change: override the default description of nserror (in registry client only) and produce a more user friendly diagnostics

@tomerd
Copy link
Contributor Author

tomerd commented Mar 25, 2023

@swift-ci smoke test

1 similar comment
@tomerd
Copy link
Contributor Author

tomerd commented Mar 25, 2023

@swift-ci smoke test

tomerd added 2 commits March 27, 2023 10:09
motivation: improved user experience

change: override the default description of nserror and produce a more user friendly diagnostics
@tomerd
Copy link
Contributor Author

tomerd commented Mar 27, 2023

@swift-ci smoke test

@tomerd
Copy link
Contributor Author

tomerd commented Mar 27, 2023

@swift-ci smoke test windows

@tomerd tomerd self-assigned this Mar 27, 2023
@MaxDesiatov
Copy link
Contributor

@swift-ci test windows

@tomerd
Copy link
Contributor Author

tomerd commented Mar 27, 2023

windows failure unrelated - @compnerd aware

@tomerd tomerd merged commit 76a2d3b into swiftlang:main Mar 27, 2023
@compnerd
Copy link
Member

CC: @etcwilde

tomerd added a commit to tomerd/swift-package-manager that referenced this pull request Mar 27, 2023
motivation: improved user experience

change: override the default description of nserror and produce a more user friendly diagnostics
tomerd added a commit that referenced this pull request Mar 27, 2023
motivation: improved user experience

change: override the default description of nserror and produce a more user friendly diagnostics
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.

4 participants