-
Notifications
You must be signed in to change notification settings - Fork 143
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
Use auto generation for DotnetToolSettings.xml #1764
Comments
My attempts at implementing this task failed, so I am taking out this task of the First testsSimply adding I have first considered a couple of workarounds:
This is because both workarounds above only work with I have then tried four different approaches:
Approach 1I tried to change the packing command from the However, the The following task:
Generates the following issue:
Using
These issues come from the differences between Support of nuspec-only packing by Approach 2Upgrading to the latest version of nuget (i.e. changing
However, the resulting Approach 3I have tried to switch to
For reference, the command executed by
Unfortunately, I could not make it work, even locally: the command line above works, but the resulting package is empty (no
Approach 4Same result as approach 2: inverting the condition doesn't make any difference. |
This file is generated by the SDK when you set PackAsTool to true, and normally, you do not need to create it yourself. Source: https://natemcmaster.com/blog/2018/05/12/dotnet-global-tools/
Since we decided to drop support for netcore2.1, we should simplify the script by removing the file.
See: https://github.com/SonarSource/sonar-scanner-msbuild/blob/master/nuspec/netcoreglobaltool/dotnet-sonarscanner.nuspec#L32-L34
See the comment below for more info on previous attempts.
The text was updated successfully, but these errors were encountered: