FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

C#: Keep only one framework reference nuget package in standalone by tamasvajk · Pull Request #14707 · github/codeql · GitHub

/ codeql Public

C#: Keep only one framework reference nuget package in standalone - #14707

Merged
tamasvajk merged 2 commits into
github:mainfrom
tamasvajk:standalone/framework-nuget-1
Nov 8, 2023
Merged

tamasvajk merged 2 commits into
github:mainfrom
tamasvajk:standalone/framework-nuget-1

Conversation

tamasvajk commented Nov 7, 2023
edited
Loading

Copy link
Copy Markdown
Contributor

No description provided.

github-actions Bot added the C# label Nov 7, 2023

tamasvajk commented Nov 8, 2023
edited
Loading

Copy link
Copy Markdown
Contributor Author

The DCA analysis shows compilation error increases in:

  • luciana-pereira/fhir-server - this seems to be transient, see https://github.com/github/codeql-csharp-team/issues/397
  • OpenKH/OpenKh - this seems to be a valid issue. This repo targets netstandard2.0, net6.0, net472. There's a 22% increase in compilation errors.
  • akkadotnet/akka.net, 5% increase
  [001] Found .NET Core/Framework DLLs in NuGet packages at /tmp/GitHub/packages/43e4457282c3f9fa/microsoft.netcore.app.ref. Not adding installation directory.
  [001] Removed reference /tmp/GitHub/packages/43e4457282c3f9fa/microsoft.netframework.referenceassemblies.net471/1.0.3
  [001] Removed reference /tmp/GitHub/packages/43e4457282c3f9fa/netstandard.library/2.0.3/lib/netstandard1.0
  • NLog/NLog, 9% increase.
  [001] Found .NET Core/Framework DLLs in NuGet packages at /tmp/GitHub/packages/43e4457282c3f9fa/microsoft.netcore.app.ref. Not adding installation directory.
  [001] Removed reference /tmp/GitHub/packages/43e4457282c3f9fa/microsoft.netframework.referenceassemblies.net472/1.0.3
  [001] Removed reference /tmp/GitHub/packages/43e4457282c3f9fa/netstandard.library/2.0.3/lib/netstandard1.0

tamasvajk marked this pull request as ready for review November 8, 2023 10:06
tamasvajk requested a review from a team as a code owner November 8, 2023 10:06

michaelnebel left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Excellent! 👍

Yes, so this happens exactly when different target frameworks are used in different project files (if it was in the same file the assets logic would "catch" this and only use one)?

tamasvajk merged commit 4ae35d1 into github:main Nov 8, 2023
tamasvajk deleted the standalone/framework-nuget-1 branch November 8, 2023 12:29
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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL