Closed Jericho closed 6 years ago
@Jericho Thanks for the heads up. Am I correct in assuming these are the best-practices you are talking about?
Yes it is. Although, to be honest with you, I don't think the page is fully up to date. For instance, it's still talks about 0.22.0 being the most recent version of Cake and it also talks about IL merging your dependencies which is no longer necessary.
We performed an automated audit of your Cake addin and found that it does not follow all the best practices.
We encourage you to make the following modifications:
.csproj
should have a line similar to this:<PackageReference Include="Cake.Core" Version="0.28.0" PrivateAssets="All" />
.csproj
should have a line similar to this:<PackageReference Include="Cake.Common" Version="0.28.0" PrivateAssets="All" />
.csproj
should have a line like this:<PackageIconUrl>https://cdn.rawgit.com/cake-contrib/graphics/a5cf0f881c390650144b2243ae551d5b9f836196/png/cake-contrib-medium.png</PackageIconUrl>
.Apologies if this is already being worked on, or if there are existing open issues, this issue was created based on what is currently published for this package on NuGet.org and in the project on github.