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
Fix MediaBrowsers.Providers analysis issues #6335
Fix MediaBrowsers.Providers analysis issues #6335
Conversation
Thank you for the PR. If you continue making these fixes, please split the PR up into parts so reviewing is easier |
How would you like me to do that? Like 20 fixes at a time or number of files? |
How I typically organize it is to remove x number of warnings, each warning in a commit see #6281 for example |
Oh, I see. I never thought about doing it that way. I see I have 24 more errors to fix for the build to pass. If it's OK, I'll do fix those, and then switch to your approach for the next one. |
That works for me, thanks again |
MediaBrowser.Providers/Plugins/MusicBrainz/MusicBrainzAlbumProvider.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Bond-009 <bond.009@outlook.com>
I believe I resolved the outstanding feedback. Thanks! |
Resolves all analysis issues for MediaBrowsers.Providers project.
Supports #2149