Azure / Azure/ResourceModules

[PSRule] Run PSRule pre-flight validation on diff (PR check)

Open
#2,150 4 comments 0 reactions 0 assignees View on GitHub
[cat] github [cat] pipelines [cat] testing enhancement
Dominant language
PowerShell
Stars
737
Forks
436
Avg merge
10d 7h
Merged PRs (30d)
1

Description

CARML already has linter and broken links checks running when pull requests are open. The check is not blocking for the PR to be merged.
> Note: pull_requests trigger works only for PRs opened from branches, not from forks.

This issue is about extending the PR checks to also run PSRule pre-flight validation on diff (on the code changed by the PR).
As a first step, the suggestion is to keep the check non-blocking, allowing PRs to be merged even if they fail PSRule checks.

PR #2094 hosts a PoC running on the whole library. That should be updated to run only on diff.

Collect feedback on the PR and agree on possible missing implementation before merge.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.