Skip to content

🚀 [Feature]: Add settings control of repository linter#218

Merged
Marius Storhaug (MariusStorhaug) merged 25 commits intomainfrom
settings
Oct 5, 2025
Merged

🚀 [Feature]: Add settings control of repository linter#218
Marius Storhaug (MariusStorhaug) merged 25 commits intomainfrom
settings

Conversation

@MariusStorhaug
Copy link
Member

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Oct 5, 2025

The workflow now provides fine-grained control over repository linting behavior through simple configuration options. You can disable the linter entirely or customize which validations run by configuring super-linter environment variables directly in your settings file, giving you full control over code quality checks without modifying workflow files.

Linter Configuration

Added two new settings to control repository linting behavior:

  • Linter.Skip: Completely disable repository linting when set to true
  • Linter.env: Configure super-linter environment variables to customize which validations run

What you need to do: Add these settings to your .github/PSModule.yml file as needed.

Disable the linter completely

Linter:
  Skip: true

Customize specific validations

Linter:
  env:
    VALIDATE_BIOME_FORMAT: false
    VALIDATE_JSCPD: false
    VALIDATE_JSON_PRETTIER: false

Advanced configuration

Linter:
  env:
    LOG_LEVEL: DEBUG
    FILTER_REGEX_EXCLUDE: '.*test.*'

See the super-linter environment variables documentation for all available options.

…mic environment variables; remove obsolete PSModule configuration files.
Copilot AI review requested due to automatic review settings October 5, 2025 16:30
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 pull request centralizes linter configuration management by moving environment variables from the workflow file to a YAML manifest and introducing dynamic loading capabilities. The change enables more flexible control of repository linter settings through configuration files rather than hard-coded workflow values.

Key changes:

  • Added dynamic environment variable loading from YAML manifest using PowerShell
  • Centralized linter configuration in PSModule.yml manifest file
  • Removed legacy configuration files (PSModule.json and PSModule.psd1)

Reviewed Changes

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

File Description
.github/workflows/workflow.yml Added PowerShell as default shell and dynamic environment loading step for linter configuration
tests/srcWithManifestTestRepo/.github/PSModule.yml Added Linter.env section with environment variables for linter control
tests/srcWithManifestTestRepo/.github/PSModule.psd1 Removed legacy PowerShell configuration file
tests/srcWithManifestTestRepo/.github/PSModule.json Removed legacy JSON configuration file

…set environment variables; update documentation accordingly.
…move skip option for Linter in PSModule configuration.
Copilot AI review requested due to automatic review settings October 5, 2025 16:42
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 6 out of 6 changed files in this pull request and generated 1 comment.

Copilot AI review requested due to automatic review settings October 5, 2025 16:49
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 6 out of 6 changed files in this pull request and generated 1 comment.

Copilot AI review requested due to automatic review settings October 5, 2025 17:04
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 6 out of 6 changed files in this pull request and generated 2 comments.

…yle descriptions and labels based on change types
…tion instead

- Update /specify, /plan, /tasks, and /implement prompts to detect fork mode via git remote -v

- Remove all references to .fork-info.json configuration file

- Simplify fork contribution workflow with automatic detection

- Add comprehensive linter configuration documentation to README
Copilot AI review requested due to automatic review settings October 5, 2025 18:21
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 11 out of 11 changed files in this pull request and generated no new comments.

- Add instruction to check ALL changes in the branch using git diff
- Ensure Docs label is only applied when ALL changes are documentation
- Clarify that mixed changes should be classified by code changes, not docs
- Use git diff origin/main...HEAD to analyze complete branch changes
Copilot AI review requested due to automatic review settings October 5, 2025 18:42
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 11 out of 11 changed files in this pull request and generated 2 comments.

Copilot AI review requested due to automatic review settings October 5, 2025 18:51
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 11 out of 11 changed files in this pull request and generated 2 comments.

Copilot AI review requested due to automatic review settings October 5, 2025 19:04
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 11 out of 11 changed files in this pull request and generated 1 comment.

- Updated /pr prompt to use consistent ordered list structure
- PR descriptions now follow: 1) summary paragraph, 2) issue links, 3) detailed sections
- Added clear guidance for release note formatting
- Included example structure to illustrate proper format

Fixes #218
Copilot AI review requested due to automatic review settings October 5, 2025 19:18
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 11 out of 11 changed files in this pull request and generated 1 comment.

Copilot AI review requested due to automatic review settings October 5, 2025 19:37
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 12 out of 12 changed files in this pull request and generated 2 comments.

Copilot AI review requested due to automatic review settings October 5, 2025 19:46
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 12 out of 12 changed files in this pull request and generated 1 comment.

@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit 9097aa2 into main Oct 5, 2025
64 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🩹 [Patch]: Add settings for the included linter

1 participant

Comments