🩹 [Patch]: Request confirmation before removing/deleting resources where recovery is difficult/impossible (#465)

This pull request updates multiple PowerShell functions to enhance user
safety by adding a `ConfirmImpact = 'High'` attribute to the
`[CmdletBinding]` declaration for commands that perform destructive
operations. This ensures users are explicitly prompted before executing
high-impact actions. Additionally, some minor formatting adjustments are
made.

- Fixes #184

### Adjustments to `ConfirmImpact` values:

* Updated high-impact function to prompt for confirmation
(`ConfirmImpact = 'High'`).
* Removed `ConfirmImpact` attribute for functions that have low impact.

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: MariusStorhaug <[email protected]>

Author: Copilot

src/functions/public/Artifacts/Remove-GitHubArtifact.ps1

@@ -1,4 +1,4 @@
-function Remove-GitHubArtifact {
+function Remove-GitHubArtifact {
     <#
         .SYNOPSIS
         Deletes an artifact from a GitHub repository by its unique ID.
@@ -31,7 +31,7 @@
         [Delete an artifact](https://docs.github.com/rest/actions/artifacts#delete-an-artifact)
     #>
     [OutputType([GitHubArtifact])]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The account owner of the repository. The name is not case sensitive.
         [Parameter(Mandatory, ValueFromPipelineByPropertyName)]

src/functions/public/Environments/Remove-GitHubEnvironment.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubEnvironment {
+filter Remove-GitHubEnvironment {
     <#
         .SYNOPSIS
         Deletes an environment from a repository.
@@ -19,7 +19,7 @@
         [Delete environments](https://docs.github.com/rest/deployments/environments?#delete-an-environment)
     #>
     [OutputType([void])]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The name of the organization.
         [Parameter(Mandatory, ValueFromPipelineByPropertyName)]

src/functions/public/Releases/Assets/Remove-GitHubReleaseAsset.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubReleaseAsset {
+filter Remove-GitHubReleaseAsset {
     <#
         .SYNOPSIS
         Delete a release asset
@@ -20,7 +20,7 @@
         .NOTES
         [Delete a release asset](https://docs.github.com/rest/releases/assets#delete-a-release-asset)
     #>
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The account owner of the repository. The name is not case sensitive.
         [Parameter(Mandatory)]

src/functions/public/Releases/Remove-GitHubRelease.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubRelease {
+filter Remove-GitHubRelease {
     <#
         .SYNOPSIS
         Delete a release
@@ -24,7 +24,7 @@
         [Delete a release](https://docs.github.com/rest/releases/releases#delete-a-release)
     #>
     [OutputType([void])]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The account owner of the repository. The name is not case sensitive.
         [Parameter(Mandatory, ValueFromPipelineByPropertyName)]

src/functions/public/Secrets/Remove-GitHubSecret.ps1

@@ -32,7 +32,7 @@
         'PSShouldProcess', '', Scope = 'Function',
         Justification = 'This check is performed in the private functions.'
     )]
-    [CmdletBinding(DefaultParameterSetName = 'AuthenticatedUser', SupportsShouldProcess)]
+    [CmdletBinding(DefaultParameterSetName = 'AuthenticatedUser', SupportsShouldProcess, ConfirmImpact = 'High')]
     param (
         # The account owner of the repository. The name is not case sensitive.
         [Parameter(Mandatory, ParameterSetName = 'Organization', ValueFromPipelineByPropertyName)]

src/functions/public/Secrets/SelectedRepository/Remove-GitHubSecretSelectedRepository.ps1

@@ -28,7 +28,7 @@
         Justification = 'Long links'
     )]
     [OutputType([void])]
-    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'Low')]
+    [CmdletBinding(SupportsShouldProcess)]
     param(
         # The account owner of the repository. The name is not case sensitive.
         [Parameter(Mandatory)]

src/functions/public/Teams/Remove-GitHubTeam.ps1

@@ -17,7 +17,7 @@
         https://psmodule.io/GitHub/Functions/Teams/Remove-GitHubTeam
     #>
     [OutputType([void])]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The organization name. The name is not case sensitive.
         # If not provided, the organization from the context is used.

src/functions/public/Users/GPG-Keys/Remove-GitHubUserGpgKey.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubUserGpgKey {
+filter Remove-GitHubUserGpgKey {
     <#
         .SYNOPSIS
         Delete a GPG key for the authenticated user
@@ -20,7 +20,7 @@
         https://psmodule.io/GitHub/Functions/Users/GPG-Keys/Remove-GitHubUserGpgKey
     #>
     [OutputType([pscustomobject])]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The ID of the GPG key.
         [Parameter(

src/functions/public/Users/Keys/Remove-GitHubUserKey.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubUserKey {
+filter Remove-GitHubUserKey {
     <#
         .SYNOPSIS
         Delete a public SSH key for the authenticated user
@@ -21,7 +21,7 @@
     #>
     [OutputType([pscustomobject])]
     [Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSAvoidLongLines', '', Justification = 'Contains a long link.')]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The unique identifier of the key.
         [Parameter(

src/functions/public/Users/SSH-Signing-Keys/Remove-GitHubUserSigningKey.ps1

@@ -1,4 +1,4 @@
-filter Remove-GitHubUserSigningKey {
+filter Remove-GitHubUserSigningKey {
     <#
         .SYNOPSIS
         Delete an SSH signing key for the authenticated user
@@ -22,7 +22,7 @@
     #>
     [OutputType([pscustomobject])]
     [Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSAvoidLongLines', '', Justification = 'Contains a long link.')]
-    [CmdletBinding(SupportsShouldProcess)]
+    [CmdletBinding(SupportsShouldProcess, ConfirmImpact = 'High')]
     param(
         # The unique identifier of the SSH signing key.
         [Parameter(