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 and remove unnecessary package dependencies #15268

Merged
merged 7 commits into from
Nov 21, 2023

Conversation

ronaldbarendse
Copy link
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

Description

While updating dependencies on our commercial products, I noticed the version of Microsoft.SourceLink.GitHub was bumped from 1.1.1 to 8.0.0 and looking at the release notes, it's now included in .NET 8 SDK and enabled by default. This means the explicit package reference can be removed and because the EmbedUntrackedSources property is now also enabled by default (see SDK target), so that one could be cleaned up as well.

I've also removed the AngleSharp package reference from Umbraco.Tests.UnitTests, since only a custom GetItemByIndex() extension method from that library was used (which is easily replaced by ElementAt().

Finally, the following package dependencies have also been updated (to versions that support .NET 8):

Testing can be done by inspecting whether the NuGet packages still contain the GitHub repository URL and separate snupkg symbol packages are created.

@ronaldbarendse ronaldbarendse merged commit 524b5ef into release/13.0 Nov 21, 2023
@ronaldbarendse ronaldbarendse deleted the v13/improvement/update-dependencies branch November 21, 2023 12:12
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.

2 participants