I'd like to suggest adding some documentation, since this took me some time to figure out.
The Windows implementation for showing a notification requires Windows PowerShell, since I assume there is windows-specific UI code required to do that kind of thing. So this module can't be placed in the standard module locations for PowerShell 7/Core. It needs to be placed in C:\Windows\System32\WindowsPowerShell\v1.0\Modules. I think this should be spelled out, since the "cross-platform" description made me assume otherwise. This still doesn't get auto-loaded properly though, because the manifest says that this module works with Core. So PowerShell loads the module directly instead of using the compatibility layer.
In short: on Windows, this module needs to be downloaded to C:\Windows\System32\WindowsPowerShell\v1.0\Modules and the manifest file needs to be edited to remove Core from CompatiblePSEditions to work properly with PowerShell 7/Core. I could submit a PR with this added to the readme, but was wondering if maybe I misunderstand something and there could be a discussion on ways to improve this.
I'd like to suggest adding some documentation, since this took me some time to figure out.
The Windows implementation for showing a notification requires Windows PowerShell, since I assume there is windows-specific UI code required to do that kind of thing. So this module can't be placed in the standard module locations for PowerShell 7/Core. It needs to be placed in
C:\Windows\System32\WindowsPowerShell\v1.0\Modules. I think this should be spelled out, since the "cross-platform" description made me assume otherwise. This still doesn't get auto-loaded properly though, because the manifest says that this module works with Core. So PowerShell loads the module directly instead of using the compatibility layer.In short: on Windows, this module needs to be downloaded to
C:\Windows\System32\WindowsPowerShell\v1.0\Modulesand the manifest file needs to be edited to removeCorefromCompatiblePSEditionsto work properly with PowerShell 7/Core. I could submit a PR with this added to the readme, but was wondering if maybe I misunderstand something and there could be a discussion on ways to improve this.