Skip to content

🩹 [Patch]: Update linter and git configuration #97

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

Merged
merged 3 commits into from
Feb 16, 2025
Merged

Conversation

MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Feb 16, 2025

Description

This pull request includes several updates to configuration files and scripts, primarily focusing on code quality and compliance improvements. The most significant changes include updating linter configurations, modifying PowerShell script settings, and updating the license year.

Linter Configuration Updates:

  • Added a new .jscpd.json configuration file to set up code duplication detection with specific thresholds and ignore patterns.
  • Updated .powershell-psscriptanalyzer.psd1 to enable specific rules and adjust settings for PowerShell script analysis.
  • Updated .github/workflows/Linter.yml to disable JSON Prettier validation.

PowerShell Script Updates:

  • Added SuppressMessageAttribute to suppress specific PSScriptAnalyzer rules in Resolve-PSModuleDependency.ps1 and Test-PSModule.ps1. [1] [2]
  • Added SuppressMessageAttribute to the main script main.ps1 to suppress the PSAvoidUsingWriteHost rule.

License Update:

  • Updated the license year in the LICENSE file from 2024 to 2025.

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

@MariusStorhaug MariusStorhaug merged commit 1fd648f into main Feb 16, 2025
10 checks passed
@MariusStorhaug MariusStorhaug deleted the fix branch February 16, 2025 17:17
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