sensu-plugins / sensu-plugins-windows

Sensu Windows Plugins
http://sensu-plugins.io
MIT License
22 stars 46 forks source link

Update testing to use Pester and create a `sensu-go` asset #99

Closed derekgroh closed 4 years ago

derekgroh commented 4 years ago

Pull Request Checklist

General

Purpose

Known Compatibility Issues

None

derekgroh commented 4 years ago

Scripts need to be in the bin directory to have SensuGo source them properly.

majormoses commented 4 years ago

I see, can you please call that out clearly in the changelog so that people understand why it's happening and what they need to do. We need to communicate changes clearly especially when they are breaking. Someone could still be installing Ruby and the gem outside of the context of an asset and it likely would still work as it used to. Additionally there are likely still sensu-ruby users out there relying on it's old behavior. It does not matter if you break one or all users, breaking is breaking.

Example:

### Breaking Changes
- moved all powershell scripts directly into the bin directory, you will need to update your check definition paths to reflect. This change is required to create an asset that works with sensu-go. `@derekgroh`