-
Notifications
You must be signed in to change notification settings - Fork 342
[Enhancement] Install-ChocolateyService #489
Comments
I have created some code and would like to push it to the master |
Hey, In order to get code added to the codebase, you would need to submit a pull request. Have a look here: https://github.com/chocolatey/chocolatey/blob/master/CONTRIBUTING.md For the process, and let us know if you have any questions. Gary |
Thank you. I will send you an email. |
I cannot find the mailing list. Could you indicate where this list could be found? |
Hey, No probs. The mailing list is hosted in a Google Group which you can find here: https://groups.google.com/forum/#!forum/chocolatey Gary |
Thank you |
I have submitted a post but cannot find it |
Hey, Since this is your first post to the forum, it was sitting in the moderation queue. I have just approved it, so you should see it now. Gary |
Yes it is visible now https://groups.google.com/forum/#!topic/chocolatey/6xq1-TIHigE. Thank you |
May I build and commit this feature this weekend? |
Hello, Have you read through this: https://github.com/chocolatey/chocolatey/blob/master/CONTRIBUTING.md This is the process that we follow for accepting contributions to Chocolatey. You would need to make the changes, and then submit the changes in the form of a Pull Request. Gary |
Thank you. I am now creating and running tests using Pester. When I have completed the Test class I will follow the next step. Thank you for the link to the process. |
Left some comments at 9a0af45 |
Also, you can just use |
All 10 test pass, but there are still some issues:
and
I will check the latter tomorrow by running the test using the verbose parameter |
Issue:
persist. How could this be solved? |
Looks like you are using version 2.0.1 of Pester. As per the setup script here: https://github.com/chocolatey/chocolatey/blob/master/setup.ps1 And also the discussion here: https://groups.google.com/forum/#!topic/chocolatey/9oQwi0-z3K0/discussion Chocolatey needs version 2.0.2 of Pester. Can you try installing this version, and let us know how you get on? |
and subsequently executing
works. Thank you. |
Work on comments @ferventcoder has been done. The code and tests have been created. Looking forward to hear whether the code and tests need to be improved. |
@030 If you submit a PR we can work through changes there. It's much easier to do. |
@ferventcoder Thank you. Pull request (GH-506) has been created. |
Helper Function which checks whether a certain service exists, a port is available and installs a service.
The text was updated successfully, but these errors were encountered: