Skip to content

feat(MyHandle): add function to retrieve GitHub username and improve logging#55

Merged
rulasg merged 6 commits intomainfrom
improvements
Mar 12, 2026
Merged

feat(MyHandle): add function to retrieve GitHub username and improve logging#55
rulasg merged 6 commits intomainfrom
improvements

Conversation

@rulasg
Copy link
Copy Markdown
Owner

@rulasg rulasg commented Mar 12, 2026

Introduce a new function to retrieve the GitHub username and enhance the debug and verbose logging capabilities. Correct the trace condition logic to ensure accurate logging behavior.

[System.Environment]::SetEnvironmentVariable($moduleDEbugLoggingVarName, $logFilePath)
}

function get-DebugSections(){

Check warning

Code scanning / PSScriptAnalyzer

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

The cmdlet 'get-DebugSections' uses a plural noun. A singular noun should be used instead.
return $sections
}

function set-DebugSections($sections){

Check warning

Code scanning / PSScriptAnalyzer

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

Function 'set-DebugSections' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'.
return $sections
}

function set-DebugSections($sections){

Check warning

Code scanning / PSScriptAnalyzer

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

The cmdlet 'set-DebugSections' uses a plural noun. A singular noun should be used instead.
return $logfile
}

function set-LogFile($logFilePath){

Check warning

Code scanning / PSScriptAnalyzer

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

Function 'set-LogFile' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'.
[System.Environment]::SetEnvironmentVariable($moduleDEbugLoggingVarName, $logFilePath)
}

function get-VerboseSections{

Check warning

Code scanning / PSScriptAnalyzer

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

The cmdlet 'get-VerboseSections' uses a plural noun. A singular noun should be used instead.
return $sections
}

function set-VerboseSections($sections){

Check warning

Code scanning / PSScriptAnalyzer

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

Function 'set-VerboseSections' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'.
return $sections
}

function set-VerboseSections($sections){

Check warning

Code scanning / PSScriptAnalyzer

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

The cmdlet 'set-VerboseSections' uses a plural noun. A singular noun should be used instead.
function Get-MyHandle{
[CmdletBinding()]
param()

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace
param()

$user = Invoke-MyCommand -Command GetGhHandle

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace
@rulasg rulasg enabled auto-merge March 12, 2026 08:49
@rulasg rulasg disabled auto-merge March 12, 2026 08:49
@rulasg rulasg merged commit b635375 into main Mar 12, 2026
3 checks passed
@rulasg rulasg deleted the improvements branch March 12, 2026 08:49
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