Skip to content

Conversation

@rulasg
Copy link
Owner

@rulasg rulasg commented Dec 5, 2024

New functionalities added:

  • public/codespace/codespaces.ps1: Added the New-DemoCodespace function to create a new codespace in the main branch of a specified GitHub repository using the GitHub CLI (gh).
  • public/codespace/codespaces.ps1: Added the Get-DemoCodespaces function to list existing codespaces for a specified owner using the GitHub CLI (gh).

.NOTES
Requires GitHub CLI (gh) to be installed and authenticated.
#>
function New-DemoCodespace {

Check warning

Code scanning / PSScriptAnalyzer

Function 'New-DemoCodespace' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'. Warning

Function 'New-DemoCodespace' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'.
$ghCommand += " --display-name '$DisplayName'"
}

$result = Invoke-Expression $ghCommand

Check warning

Code scanning / PSScriptAnalyzer

Invoke-Expression is used. Please remove Invoke-Expression from script and find other options instead. Warning

Invoke-Expression is used. Please remove Invoke-Expression from script and find other options instead.
} Export-ModuleMember -Function 'New-DemoCodespace'


function Get-DemoCodespaces {

Check notice

Code scanning / PSScriptAnalyzer

The cmdlet 'Get-DemoCodespaces' does not have a help comment. Note

The cmdlet 'Get-DemoCodespaces' does not have a help comment.
} Export-ModuleMember -Function 'New-DemoCodespace'


function Get-DemoCodespaces {

Check warning

Code scanning / PSScriptAnalyzer

The cmdlet 'Get-DemoCodespaces' uses a plural noun. A singular noun should be used instead. Warning

The cmdlet 'Get-DemoCodespaces' uses a plural noun. A singular noun should be used instead.
@rulasg rulasg merged commit c43646e into main Dec 5, 2024
3 checks passed
@rulasg rulasg deleted the codespaces branch December 5, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants