Skip to content

Add scoped Bicep lint validation#559

Open
WilliamBerryiii wants to merge 3 commits into
mainfrom
feature/553-bicep-lint-performance-fast-path
Open

Add scoped Bicep lint validation#559
WilliamBerryiii wants to merge 3 commits into
mainfrom
feature/553-bicep-lint-performance-fast-path

Conversation

@WilliamBerryiii
Copy link
Copy Markdown
Member

Description

Adds scoped Bicep lint validation so PR validation can focus lint work on changed folders instead of running broader validation unnecessarily.

Related Issue

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • CI/CD pipeline change
  • Security improvement
  • Performance improvement
  • Refactoring

Implementation Details

  • Updates the Bicep lint workflow for scoped validation.
  • Updates matrix folder-change detection and PR validation workflow wiring.
  • Updates folder-change detection logic and test coverage in Detect-Folder-Changes.ps1.
  • Updates Bicep lint invocation logic and test coverage in Invoke-BicepLint.ps1.

Changed files:

  • .github/workflows/bicep-lint.yml
  • .github/workflows/matrix-folder-check.yml
  • .github/workflows/pr-validation.yml
  • scripts/build/Detect-Folder-Changes.ps1
  • scripts/build/Detect-Folder-Changes.Tests.ps1
  • scripts/build/Invoke-BicepLint.ps1
  • scripts/build/Invoke-BicepLint.Tests.ps1

Testing Performed

  • Updated PowerShell test coverage for folder-change detection and Bicep lint invocation.
  • CI validation is expected to run on this pull request.

Validation Steps

  1. Open a PR with Bicep-related changes and confirm scoped Bicep lint runs for the affected folders.
  2. Open a PR without Bicep-related changes and confirm unnecessary Bicep lint work is skipped.
  3. Confirm PR validation still receives the expected changed-folder outputs.

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Comments added for complex logic where needed
  • Documentation updated if required
  • Tests added or updated
  • No breaking changes introduced
  • Security considerations reviewed

Security Review

  • No secrets, tokens, credentials, or sensitive data are intentionally included.
  • Changes are limited to CI workflow logic and build/test scripts.

Additional Notes

Branch diff summary: 7 files changed, 894 insertions, 103 deletions.

@WilliamBerryiii WilliamBerryiii requested a review from a team as a code owner May 24, 2026 01:38
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.41%. Comparing base (41b933a) to head (a78d8d0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #559   +/-   ##
=======================================
  Coverage   32.41%   32.41%           
=======================================
  Files          40       40           
  Lines        5902     5902           
=======================================
  Hits         1913     1913           
  Misses       3989     3989           
Flag Coverage Δ
rust 32.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Improve Bicep lint and validation CI performance

2 participants