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

allow for differing package and assembly names when finding packages path #11064

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

brettfo
Copy link
Contributor

@brettfo brettfo commented Dec 5, 2024

Found during a manual scan of the logs.

When determining if a .csproj has an assembly reference to a package, an incorrect assumption was made that the name of the assembly would match the name of the package, but that's not correct.

The correct behavior is to match the HintPath with the package name, because that will always match. We had an existing regular expression that did the correct matching, so this was lifted higher in the class so it could be re-used.

@brettfo brettfo requested a review from a team as a code owner December 5, 2024 20:12
@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Dec 5, 2024
@randhircs randhircs merged commit c5bd1b3 into main Dec 5, 2024
70 checks passed
@randhircs randhircs deleted the dev/brettfo/nuget-null-path branch December 5, 2024 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants