Skip to content

chore: Bicep Parameter validation email format change#2212

Merged
Roopan-Microsoft merged 3 commits into
Azure-Samples:devfrom
NirajC-Microsoft:dev
May 8, 2026
Merged

chore: Bicep Parameter validation email format change#2212
Roopan-Microsoft merged 3 commits into
Azure-Samples:devfrom
NirajC-Microsoft:dev

Conversation

@NirajC-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request introduces a new, structured HTML email report for Bicep parameter validation results, and integrates it into the GitHub Actions workflow. The workflow now generates and sends visually improved email notifications with detailed validation results, replacing the previous plain-text approach. The main changes are grouped below:

Workflow and Notification Improvements:

  • The GitHub Actions workflow (.github/workflows/validate-bicep-params.yml) now generates an HTML report (email_body.html) using the validation script and attaches it as an artifact. The notification steps have been updated to send this HTML report as the email body instead of a plain-text summary. [1] [2] [3]

Validation Script Enhancements:

  • Added a new function generate_html_report and supporting code to scripts/validate_bicep_params.py to build a detailed, Outlook-compatible HTML report summarizing validation results, including error/warning counts and per-file breakdowns.
  • Extended the script’s CLI with new arguments: --html-output, --accelerator-name, and --run-url to support HTML report generation with contextual metadata.
  • The script now writes the HTML report to the specified path when the --html-output argument is provided.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit fa7aef0 into Azure-Samples:dev May 8, 2026
2 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.

2 participants