-
-
Notifications
You must be signed in to change notification settings - Fork 41
Update chocolatey package #190
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
Update chocolatey package #190
Conversation
- This is no longer required, as Chocolatey wraps this up internally
- And Install-VisualStudioVsixExtension rather than Install-ChocolateyVsixPackage - This will mean that package installation will work for VS 2017, as well as older versions
@laurentkempe I have started a PR for the suggested changes to the Chocolatey Package for GitDiffMargin. This includes the usage of the Chocolatey Extension Package for Visual Studio: https://chocolatey.org/packages/chocolatey-visualstudio.extension This includes an API compatible replacement of the built-in helper method: https://chocolatey.org/docs/helpers-install-chocolatey-vsix-package Which also works for newer versions of Visual Studio. In addition to these changes, I would also suggest that you include the Checksum and Checksum Type parameters to the Cmdlet. This will allow Chocolatey to verify that the vsix that is downloaded at installation time, matches what you expect it to be. You should be able to add a step into your release pipeline to update there values, once you know what they are i.e. when the build is completed. |
Thx @gep13 for the PR, very much appreciated! |
@laurentkempe Yes, that is certainly an option, and it does result in the most stable of packages, due to the fact that everything that is needed to do the installation is contained within the package. If you go down this route, you will need to provide a couple of extra files into your package, namely the VERIFICATION.TXT and LICENSE.TXT file. If you run the |
@gep13 Ok I will try that |
@laurentkempe let me know if you have any questions. |
@gep13 Do you know one VS extension which is doing it like that? So I could have a look to it |
@laurentkempe looks like this package is doing it: https://github.com/chocolatey/chocolatey-coreteampackages/tree/master/automatic/sonarlint-vs2015 NOTE: That package is using the built-in cmdlet that ships with Chocolatey, and this isn't the one that you want to be using, so you will need to change that part. The remainder of the installation script should be valid though. |
4c38744
to
de01aff
Compare
de01aff
to
3644647
Compare
@gep13 I have the build which work and build this artifact - GitDiffMargin.3.9.2.47.nupkg.zip When I follow the instruction to test the package I get the following error:
Do you have any clue what could be the problem? |
@laurentkempe was VS open at the time when you tried to install the package? If so, can you try closing it, and running the install again? |
@gep13 yeah that was the issue! 🎉 So next step will be to have my release pipeline doing a choco publish 😸 |
@laurentkempe sounds great! |
@gep13 👍 x 💯 Thanks a lot for the support and your great extensions for Azure Devops! |
No description provided.