Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor update and fix pre-merge script block #238

Open
wants to merge 6 commits into
base: testing
Choose a base branch
from

Conversation

SamErde
Copy link
Collaborator

@SamErde SamErde commented Mar 5, 2025

Updated the pre-merge script block to include the cmdletbinding attribute and full parameter definitions that are already present in Invoke-Locksmith.

Added PSGallery tags to the module manifest after seeing additional tags used by other ADCS/PKI related projects.

Updated the version of Jinja used by MkDocs to the latest version that closes a security vulnerability.


Updates to the Locksmith module build script:

  • Build/Build-Module.ps1: Updated the Tags property to include additional relevant keywords.
  • Build/Build-Module.ps1: Added CmdletBinding attribute and additional parameters with default values to the $PreScriptMerge script block, including a HelpUri and descriptions for Mode and Scans parameters.

Dependency version update:

@SamErde SamErde requested a review from jakehildreth March 5, 2025 02:18
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.

1 participant