Skip to content

Conversation

@MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Jan 28, 2026

This pull request primarily updates PowerShell scripts and workflow configuration to improve code consistency and maintainability. The main changes focus on standardizing string casing and operator usage, correcting minor typos, and updating the GitHub Actions linter version.

PowerShell Script Improvements:

  • Standardized the use of -replace and -split operators to lowercase in several scripts.
  • Updated the conditional operator from -NotIn to -notin in Build-PSModuleManifest.ps1 for proper PowerShell syntax.

Typo and Comment Corrections:

  • Fixed typos in comments and attribute justifications, such as changing "compatability" to "compatibility" and "doesnt" to "does not" in Build-PSModuleManifest.ps1 and Build-PSModuleRootModule.ps1.

Workflow Configuration Update:

  • Updated the linter action in .github/workflows/Linter.yml to use super-linter version 8.4.0.

Linter Configuration:

  • Added .codespellrc configuration file for codespell linter.

@MariusStorhaug MariusStorhaug marked this pull request as ready for review January 28, 2026 18:38
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner January 28, 2026 18:38
Copilot AI review requested due to automatic review settings January 28, 2026 18:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the super-linter GitHub Action from v8.3.2 to v8.4.0 and enables the new SUPPRESS_OUTPUT_ON_SUCCESS feature to reduce CI log noise. Additionally, several spelling errors identified by the linter have been corrected.

Changes:

  • Updated super-linter action to v8.4.0 with SHA hash verification
  • Enabled SUPPRESS_OUTPUT_ON_SUCCESS to show only failed linter output
  • Fixed spelling errors: "doesnt" → "does not", "compatability" → "compatibility", "environemnt" → "environment", "pacakge" → "package"

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/Linter.yml Updated super-linter from v8.3.2 to v8.4.0 and added SUPPRESS_OUTPUT_ON_SUCCESS environment variable
src/helpers/Build/Build-PSModuleRootModule.ps1 Fixed spelling in justification comments: "doesnt" → "does not"
src/helpers/Build/Build-PSModuleManifest.ps1 Fixed spelling in comment: "compatability" → "compatibility"
.github/linters/.textlintrc Fixed spelling in textlint rules: "environemnt" → "environment" and "pacakge" → "package"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings January 28, 2026 20:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings January 28, 2026 21:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MariusStorhaug MariusStorhaug force-pushed the update/super-linter-v8.4.0 branch from d3286d0 to 2ab8a73 Compare January 28, 2026 22:10
Copilot AI review requested due to automatic review settings January 28, 2026 22:35
@MariusStorhaug MariusStorhaug force-pushed the update/super-linter-v8.4.0 branch from 2ab8a73 to 9ffa60b Compare January 28, 2026 22:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

2 participants