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

Update support libraries (and drop QueryException) #57

Merged
merged 8 commits into from
Dec 29, 2023

Conversation

Zastai
Copy link
Owner

@Zastai Zastai commented Dec 29, 2023

This updates MetaBrainz.Build.Sdk to v3.1.2, MetaBrainz.Common to v3.0.0 and MetaBrainz.Common.Json to v6.0.1.

QueryException was dropped in favour of the HttpError from MetaBrainz.Common.

`MusicBrainz.Common.HttpError` is used instead.
All methods that issue a web request are now properly documented as
potentially throwing `HttpRequestException` or `HttpError`.
These include:

- use of `is [not] null` instead of `==/!= null`
- expression-valued method bodies
In the case of `DiskIdLookupResult`, they were replaced by specific
constructors; in all other cases they were changed from `set` to `init`.
@Zastai Zastai added dependencies Updates to dependencies breaking An incompatible change to a feature or API (requires a bump of the major version) nuget Version updates for NuGet packages labels Dec 29, 2023
@Zastai Zastai self-assigned this Dec 29, 2023
@Zastai Zastai merged commit 157c6bf into main Dec 29, 2023
2 checks passed
@Zastai Zastai deleted the update-support-libraries branch December 29, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking An incompatible change to a feature or API (requires a bump of the major version) dependencies Updates to dependencies nuget Version updates for NuGet packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant