Skip to content

Conversation

@rulasg
Copy link
Owner

@rulasg rulasg commented Jan 20, 2026

Refactor the include helpers modules to improve structure and functionality. Changes include updating parameter handling, modifying function names for clarity, and enhancing the management of feature flags. This update aims to streamline the codebase and improve maintainability.

# function Get-ModuleNameRegisteredFeatureFlags{
# [cmdletbinding()]
# param()
function Get-ModuleNameRegisteredFeatureFlags{

Check warning

Code scanning / PSScriptAnalyzer

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

The cmdlet 'Get-ModuleNameRegisteredFeatureFlags' uses a plural noun. A singular noun should be used instead.
if ([string]::IsNullOrEmpty($ModulePath)) {
$modulePath = $MODULE_PATH | Split-Path -Parent
}

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace

if([string]::IsNullOrWhiteSpace($ModuleName)){
$moduleName = $MODULE_NAME
}

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace
if([string]::IsNullOrWhiteSpace($ModuleName)){
$moduleName = $MODULE_NAME
}

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace
}

# $Json = Get-Content $ffPath
$Json = Get-Content $ffPath

Check notice

Code scanning / PSScriptAnalyzer

Line has trailing whitespace Note

Line has trailing whitespace
@rulasg rulasg merged commit 8e49d54 into main Jan 20, 2026
3 of 5 checks passed
@rulasg rulasg deleted the v3.3.1-preview branch January 20, 2026 08:11
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