-
Notifications
You must be signed in to change notification settings - Fork 144
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
Moved from WebClient to HttpClient #841
Conversation
6d1b0f9
to
3fa7334
Compare
Kudos, SonarCloud Quality Gate passed!
|
<HintPath>..\..\packages\System.IO.4.3.0\lib\net462\System.IO.dll</HintPath> | ||
<Private>True</Private> | ||
<Private>True</Private> | ||
</Reference> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The duplicate Private attribute is an accident, isn't it?
@@ -23,6 +23,6 @@ namespace SonarQube.Scanner.MSBuild | |||
internal static class Program | |||
{ | |||
private static int Main(string[] args) | |||
=> SonarScanner.MSBuild.Program.ExecuteFromLegacyEntryPoint(args); | |||
=> SonarScanner.MSBuild.Program.ExecuteFromLegacyEntryPoint(args).Result; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about using async Main
instead?
0033af9
to
de43589
Compare
38033dd
to
f315613
Compare
Kudos, SonarCloud Quality Gate passed!
|
* Moved from WebClient to HttpClient. (#841) * Split TFS-specific code in a separate exe (#896) * Update license headers (#895) * Detect and take VsTestToolsInstallerInstalledToolLocation environment variable to set CodeCoverage.exe path. (#903) * Remove/Changed SonarQube-only log messages to be more generic with So… (#898) * Remove/Changed SonarQube-only log messages to be more generic with SonarCloud. * Fail fast if sonar.organization is provided in SonarQube.Analysis.xml (#901) * Changed deprecated projectKey parameters while fetching Quality Profile (#899) * Added .NET 5 flavor for the Scanner (#884) * Drop support for single-valued "sonar.cs.roslyn.reportFilePath" and "… (#902) * Drop support for single-valued "sonar.cs.roslyn.reportFilePath" and "sonar.cs.analyzer.projectOutPath" (and VB.Net siblings) * Add support for pure .NET Core Projects (#887) * If a ProjectGuid has not been found in either the csproj or the solution, generate a random one. * Fail fast in case the server license is invalid. (#907) * fixed bug - skipped shared across projects files (#876) * Update documentation (Proxy, .NET Core NET Version, added back conditional blocks for SQ/SC (#905) * Add a warning and clearer message in case of falling back to SonarOutputDir projectbBaseDir. (#908) * Add build configuration to TFS build
so that DefaultProxy takes env variables to be configured automatically. + moved to async for many method, and removed a codesmell for a method overlapping another one.
fixes #552
fixes #654