Skip to content

Add linting and link testing#34

Merged
lvlindv merged 20 commits intoAzure:mainfrom
jfaurskov:codereview
Apr 9, 2026
Merged

Add linting and link testing#34
lvlindv merged 20 commits intoAzure:mainfrom
jfaurskov:codereview

Conversation

@jfaurskov
Copy link
Copy Markdown
Collaborator

@jfaurskov jfaurskov commented Mar 24, 2026

Add pipeline and linters for link testing and linting
Update files with syntax errors
Configured PsscriptAnalyzerSettings.psd1

Copilot AI review requested due to automatic review settings March 24, 2026 15:36
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

Adds a GitHub Actions workflow to run automated linting and markdown link checks on pull requests, along with repo-level linter configuration files to standardize lint behavior.

Changes:

  • Add a code-review GitHub Actions workflow running Super-Linter and Linkspector on PRs to main
  • Add MarkdownLint and YAMLlint configuration under .github/linters/
  • Add Linkspector configuration under .github/actions-config/

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/code-review.yml New CI workflow for linting and link checking on PRs
.github/linters/.yaml-lint.yml YAML lint rule configuration (line length, etc.)
.github/linters/.markdown-lint.yml Markdown lint rule configuration
.github/actions-config/.linkspector.yml Linkspector scan configuration

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

Comment thread .github/workflows/code-review.yml Outdated
Comment thread .github/linters/.markdown-lint.yml Outdated
Comment thread .github/workflows/code-review.yml
Comment thread .github/linters/.yaml-lint.yml Outdated
Comment thread .github/workflows/code-review.yml Outdated
Comment thread .github/workflows/code-review.yml Outdated
jfaurskov and others added 19 commits March 26, 2026 13:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jfaurskov jfaurskov changed the title initial commit Add linting and link testing Apr 9, 2026
@jfaurskov jfaurskov requested a review from lvlindv April 9, 2026 08:58
@lvlindv lvlindv merged commit 20e53ac into Azure:main Apr 9, 2026
4 checks passed
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.

3 participants