Skip to content

chore: Bicep Parameter validation email format change#427

Merged
Roopan-Microsoft merged 4 commits into
microsoft:devfrom
NirajC-Microsoft:dev
May 8, 2026
Merged

chore: Bicep Parameter validation email format change#427
Roopan-Microsoft merged 4 commits into
microsoft:devfrom
NirajC-Microsoft:dev

Conversation

@NirajC-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request enhances the Bicep parameter validation workflow by generating a structured HTML email report and updating the notification mechanism to use this new format. The main improvements are the addition of HTML report generation in the validation script, passing this report through the workflow, and updating notification steps to send the HTML content instead of plain text.

HTML Email Reporting and Notification Improvements:

  • Added a new generate_html_report function in scripts/validate_bicep_params.py to create a detailed, styled HTML report of validation results, including error/warning summaries and workflow links.
  • Extended the CLI in validate_bicep_params.py to accept --html-output, --accelerator-name, and --run-url arguments, enabling output of the HTML report and customization of its header/footer.
  • Updated the GitHub Actions workflow (.github/workflows/validate-bicep-params.yml) to:
    • Generate the HTML report during the validation step and pass relevant environment variables.
    • Upload the generated email_body.html as an artifact.
    • Use the HTML report as the body for both failure and success notification emails, replacing the previous plain text approach. [1] [2]

These changes make validation notifications more readable and professional, improving the developer experience and clarity of automated reports.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 3 comments.

Comment thread .github/workflows/validate-bicep-params.yml
Comment thread .github/workflows/validate-bicep-params.yml
Comment thread scripts/validate_bicep_params.py Outdated
@Roopan-Microsoft Roopan-Microsoft merged commit 0bdec21 into microsoft:dev May 8, 2026
5 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.8.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants