Skip to content

🪲 [Fix]: Fix linter settings and docs#8

Merged
Marius Storhaug (MariusStorhaug) merged 7 commits into
mainfrom
fixDocs
Oct 11, 2025
Merged

🪲 [Fix]: Fix linter settings and docs#8
Marius Storhaug (MariusStorhaug) merged 7 commits into
mainfrom
fixDocs

Conversation

@MariusStorhaug
Copy link
Copy Markdown
Member

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

This pull request introduces configuration updates to our linting setup, primarily focusing on disabling certain validations and adjusting markdown linting rules to better fit our documentation conventions.

Linter configuration updates:

  • Added a Linter section to .github/PSModule.yml to explicitly disable several validation checks, including Biome format/lint, GitHub Actions Zizmor, JSCPD, and various Prettier validations for JSON, Markdown, and YAML.

Markdown linting rule adjustments:

  • Updated .github/linters/.markdown-lint.yml to disable the MD024 rule (no duplicate headings), allowing INPUTS and OUTPUTS to be repeated, and made a minor formatting change to the MD026 punctuation list.

Copilot AI review requested due to automatic review settings October 6, 2025 22:58
Copy link
Copy Markdown
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 fixes linter settings by disabling specific validation rules in the GitHub Actions configuration. The changes address linter issues by turning off various code formatting and validation checks.

  • Disables multiple linter validation rules including Biome, GitHub Actions Zizmor, JSCPD, and Prettier formatters
  • Adds a new Linter section to the PSModule configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/PSModule.yml
Copilot AI review requested due to automatic review settings October 8, 2025 09:47
Copy link
Copy Markdown
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 2 out of 2 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/linters/.markdown-lint.yml
Copilot AI review requested due to automatic review settings October 10, 2025 14:45
Copy link
Copy Markdown
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.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/functions/public/ConvertTo-UriQueryString.ps1 Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 11, 2025 06:21
Copy link
Copy Markdown
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 4 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/functions/public/New-Uri.ps1
Comment thread src/functions/public/Test-Uri.ps1
Comment thread src/functions/public/Get-Uri.ps1
Comment thread src/functions/public/ConvertTo-UriQueryString.ps1
@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit 70d9d48 into main Oct 11, 2025
38 checks passed
@github-actions
Copy link
Copy Markdown

Module Uri - 1.1.4 published to the PowerShell Gallery.

@github-actions
Copy link
Copy Markdown

GitHub release for Uri v1.1.4 has been created.

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.

2 participants