v4.0.2
🩹 [Patch]: Remove initialization step and dependency on Utilities
(#114)
Description
This pull request simplifies the codebase by removing redundant module dependencies and improving code readability across various scripts. It also consolidates file exclusions in the .jscpd.json
configuration and updates logic for handling module manifest properties.
Module Dependency Removal:
- Removed
#Requires
statements for unused module dependencies (GitHub
,Utilities
,Hashtable
) across multiple scripts, includingBuild-PSModule.ps1
,Build-PSModuleManifest.ps1
, andBuild-PSModuleRootModule.ps1
files. [1] [2] [3] [4]
Configuration Updates:
- Consolidated file exclusions in
.github/linters/.jscpd.json
by replacing specific file paths with a wildcard (**/scripts/helpers/**
).
Workflow Simplification:
- Removed the
Initialize-PSModule
step from theAction-Test.yml
workflow, streamlining the setup process. [1] [2] [3]
Code Readability Improvements:
- Replaced custom
IsNotNullOrEmpty
function calls with native.NET
methods like[string]::IsNullOrEmpty
for better clarity and maintainability in scripts such asBuild-PSModuleManifest.ps1
andGet-PSModuleAliasesToExport.ps1
. [1] [2] [3] [4]
Suppression Attribute Adjustments:
- Updated
Diagnostics.CodeAnalysis.SuppressMessageAttribute
to includeScope = 'Function'
for consistency in scripts likeGet-PSModuleClassesToExport.ps1
andGet-PSModuleCmdletsToExport.ps1
. [1] [2]
Type of change
- 📖 [Docs]
- 🪲 [Fix]
- 🩹 [Patch]
-
⚠️ [Security fix] - 🚀 [Feature]
- 🌟 [Breaking change]
Checklist
- I have performed a self-review of my own code
- I have commented my code, particularly in hard-to-understand areas