-
Notifications
You must be signed in to change notification settings - Fork 47
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
Add chocolatey management support #60
Conversation
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.
Thanks for the contribution @kartnico LGTM
@noelmcloughlin what about the tests? |
I thought about that but chocloatey only works on Windows platform. Is there any example where we have formula running on travis/windows? |
Does travis provide Windows/chocolatey testing support? Otherwise, we should try to skip those tests |
I found this: https://docs.travis-ci.com/user/reference/windows
@kartnico would you be interested in investigating Travis/Windows support for this community? This is something we are missing I think.
|
Hello, If you want to use Travis with Kitchen and Inspec on Windows, I have not found how to do it with Travis. On the other hand, other communities seem to use AppVeyor for this : https://github.com/chocolatey/chocolatey-cookbook What do you think about using AppVeyor ? Regards, Nicolas |
@myii fyi |
@noelmcloughlin maybe cirrus ci? |
This PR and specific comment is of relevance: Cirrus-CI has Windows containers available but I'm not sure how useful they will be: |
from saltstack-formulas/openvpn-formula#112 (comment)
Hi @myii could you explain the issue further particularly regarding this sentence? thanks. |
@noelmcloughlin That would be @dafyddj we'd need to get a specific response from re: saltstack-formulas/openvpn-formula#112 (comment). |
@kartnico asked a specific question too-
We probably need a dedicated issue for windows CI since it bigger scope that this PR. |
The issue here is the time it will require to do all of the research and testing. Then it would need to be discussed with the core team, to see if it can be included. If anyone wants to take that on, that would be great. In the meantime, we could at least get local testing working, based upon |
For the time being, in order for this PR to be considered for merging, the tests need to be skipped, as mentioned by @javierbertoli. @kartnico Are you able to do that? Essentially, just wrap |
The only real reference I can find on this is on Stack Overflow. |
@kartnico can you do that. |
@noelmcloughlin done ! |
Thanks @kartnico I've approved the PR but can you fix the conflicts, please? |
@kartnico Don't worry, I've fixed the conflicts and rebased this PR, including adjusting the commit messages in accordance with CC: @noelmcloughlin. |
Merged, thanks for the patience @kartnico! |
🎉 This PR is included in version 0.10.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
@kartnico The good news is that we've got Windows testing taking place in GitHub Actions for this formula now. For versions |
This PR permits to manage packages with Chocolatey Salt module here