Remove unused libraries - #272
Closed
Lauren Ciha (lauren-ciha) wants to merge 2 commits into
Closed
Conversation
Lauren Ciha (lauren-ciha)
requested review from
Copilot and
Felipe G (guimafelipe)
June 26, 2025 18:51
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR removes unused library dependencies from both the production and test projects.
- Removed unused package references from GitHubExtension.csproj (LibGit2Sharp, MessageFormat, Microsoft.Extensions.Hosting, LibGit2Sharp.NativeBinaries)
- Removed unused testing packages from GitHubExtension.Test.csproj (coverlet.collector, Microsoft.Extensions.Hosting)
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| GitHubExtension/GitHubExtension.csproj | Removed unused package references to streamline dependencies |
| GitHubExtension.Test/GitHubExtension.Test.csproj | Removed unnecessary testing packages to simplify the test setup |
Comments suppressed due to low confidence (2)
GitHubExtension/GitHubExtension.csproj:32
- Verify that any functionality previously relying on LibGit2Sharp has been completely removed or refactored prior to its removal.
<PackageReference Include="Dapper.Contrib" Version="2.0.78" />
GitHubExtension.Test/GitHubExtension.Test.csproj:12
- Removing the coverlet.collector package may affect test coverage reporting; ensure that an alternative solution is in place if coverage metrics are important.
</PropertyGroup>
Felipe G (guimafelipe)
previously approved these changes
Jun 26, 2025
Lauren Ciha (lauren-ciha)
force-pushed
the
user/laurenciha/remove-dependencies
branch
from
June 26, 2025 21:00
f2ebd69 to
b73dc20
Compare
Member
|
Lauren Ciha (@lauren-ciha) any reason why this wasn't included yet? good way to get package size down, no? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.