-
Notifications
You must be signed in to change notification settings - Fork 125
Closed
Labels
good first issueGood for newcomersGood for newcomersmaintenanceMaintenance work, no version bumpMaintenance work, no version bump
Milestone
Description
Issue Description
Validate-MarkdownFrontmatter.ps1 contains a Get-ChangedMarkdownFileGroup function that duplicates functionality already available in LintingHelpers.psm1 (Get-ChangedFilesFromGit). The script should use the shared module function instead of reimplementing git diff parsing.
Additional Context
Current state:
scripts/linting/Validate-MarkdownFrontmatter.ps1hasGet-ChangedMarkdownFileGroup(~40 lines)scripts/linting/Modules/LintingHelpers.psm1exportsGet-ChangedFilesFromGitwith similar logic
Recommended approach:
- Import
LintingHelpers.psm1at the top ofValidate-MarkdownFrontmatter.ps1 - Replace
Get-ChangedMarkdownFileGroupcalls withGet-ChangedFilesFromGitfiltered by.mdextension - Remove the duplicate function
Testing:
- Run
npm run pesterfor existing tests - Test with
npm run lint:frontmatteron changed markdown files - Verify
-ChangesOnlyparameter still works correctly
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersmaintenanceMaintenance work, no version bumpMaintenance work, no version bump