plastikfan / terminalbuddy Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 532 KB

Utility module for Windows Terminal

License: MIT License

PowerShell 100.00%

terminalbuddy's People

Contributors

Watchers

terminalbuddy's Issues

Add PlatyPS documentation

See https://github.com/PowerShell/platyPS
and also https://docs.readthedocs.io/en/latest/

Ensure global variable WindowsTerminalSettingsPath is defined properly

WindowsTerminalSettingsPath may not be avilable in the built module. Also, make sure that any reference to is subject to Test-Path, so that there are no errors if TerminlaBuddy is installed but WindowsTerminal isn't.

branch: fix-settings-path

See: https://powershell.anovelidea.org/powershell/how-i-implement-module-variables/

Make installable as module

Ensure installable as module and other tasks required for publish including documentation.

branch: prep-module-install

fix up function documentation

See https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_comment_based_help?view=powershell-7

branch: function-doc

Remove get-theme

Repace with Get-KrayolaTheme

branch: Get-KrayolaTheme

Add new function ConvertFrom-ItermColors

Create utility function to perform the extraction of the iterm colour schemes (.itermcolors files) to create themes in a Terminal compatible format.

branch: convertfrom-itermcolors

Azure Pipelines CI

Have a look at PSScriptAnalyzer on GitHub for an example of Azure CI

write-HostItemDecorator is un-necessarily wrapped

write-HostItemDecorator can easily be turned into a script block obviating the need for the wrapper object. I discovered this as a result of perusing as issue on PSScriptAnalyzer (PowerShell/PSScriptAnalyzer#711 (comment)); a comment by @rjmholt illustrates how to do this:

We can use this technique for passing write-HostItemDecorator as a parameter to invoke-ForeachFile, without powershell mistaing it for in an place invocation of the function.

Add badges

write-HostItemDecorator needs to use item-label

Currently, write-HostItemDecorator accidentally assumes that its dealing with files (when it is supposed to be more generic than that). It uses the string 'filename' in teh call to Write-ThemedPairsInColour, when this label should be defined by the client as a PassThru property called ITEM-LABEL.

It also needs to use the Product on the invoke result and write that too.
Message should not be a mandatry item.

branch: use-item-label

[+] C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\join-AllSchemas.tests.ps1 319ms 
(38ms|249ms)
Integrating new theme // ["Scheme name" => "Batman"]
[-] merge-SettingsContent.Given: content with single new scheme.Should: merge the new scheme with Settings 72ms (69ms|3ms)
 PropertyNotFoundException: The property 'schemes' cannot be found on this object. Verify that the property exists.
 at <ScriptBlock>, C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\merge-SettingsContent.tests.ps1:29
Integrating new theme // ["Scheme name" => "Batman"]
Integrating new theme // ["Scheme name" => "Highway"]
[-] merge-SettingsContent.Given: content with multiple (x2) new schemes.Should: merge the new schemes with Settings 95ms (94ms|2ms)
 PropertyNotFoundException: The property 'schemes' cannot be found on this object. Verify that the property exists.        
 at <ScriptBlock>, C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\merge-SettingsContent.tests.ps1:47
Skipping existing theme // ["Scheme name" => "Cyberdyne"]
[-] merge-SettingsContent.Given: content with single scheme already present in settings.Should: not merge new scheme 54ms (53ms|1ms)  
 PropertyNotFoundException: The property 'schemes' cannot be found on this object. Verify that the property exists.
 at <ScriptBlock>, C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\merge-SettingsContent.tests.ps1:64
Integrating new theme // ["Scheme name" => "Batman"]
Skipping existing theme // ["Scheme name" => "Cyberdyne"]
[-] merge-SettingsContent.Given: content with 1 new scheme and 1 existing scheme.Should: only merge the new scheme with Settings 60ms 
(59ms|1ms)
 PropertyNotFoundException: The property 'schemes' cannot be found on this object. Verify that the property exists.
 at <ScriptBlock>, C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\merge-SettingsContent.tests.ps1:82
Skipping existing theme // ["Scheme name" => "Cyberdyne"]
Skipping existing theme // ["Scheme name" => "Chalkboard"]
[-] merge-SettingsContent.Given: content with existing (x2) schemes.Should: not merge new scheme 54ms (52ms|2ms)
 PropertyNotFoundException: The property 'schemes' cannot be found on this object. Verify that the property exists.
 at <ScriptBlock>, C:\Users\Plastikfan\dev\github\PoSh\TerminalBuddy\Elizium.TerminalBuddy\Tests\Internal\merge-SettingsContent.tests.ps1:100
Importing Terminal Scheme // ["No" => "  1", "Scheme filename" => "Banana Blueberry.itermcolors", "Scheme name" => "Banana Blueberry"]

The following needs to be in the build script:

Set-StrictMode -Version Latest

Ensure compatibility with Elizium.Krayola 2.0

Use the scribbler.

plastikfan / terminalbuddy Goto Github PK

terminalbuddy's People

Contributors

Watchers

terminalbuddy's Issues

Recommend Projects

Recommend Topics

Recommend Org