Skip to content

feat: Spec for Output Parameter Decorators#2131

Merged
ReneHezser merged 4 commits into
mainfrom
spec-output-parameter-decorators
Jun 17, 2025
Merged

feat: Spec for Output Parameter Decorators#2131
ReneHezser merged 4 commits into
mainfrom
spec-output-parameter-decorators

Conversation

@ReneHezser

@ReneHezser ReneHezser commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

Overview/Summary

A new shared spec, which makes a description mandatory for all output parameters.

AB#41074

This PR fixes/adds/changes/removes

Breaking Changes

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@ReneHezser ReneHezser requested a review from a team as a code owner June 13, 2025 08:28
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Triage 🔍 Maintainers need to triage still label Jun 13, 2025
@microsoft-github-policy-service

Copy link
Copy Markdown

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the AVM Issue Triage documentation.

Comment thread docs/content/specs-defs/includes/bicep/shared/functional/BCPFR7.md Outdated
Comment thread docs/content/specs-defs/includes/bicep/shared/non-functional/BCPNRF23.md Outdated

@jaredfholgate jaredfholgate left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ReneHezser ReneHezser merged commit f6e12b4 into main Jun 17, 2025
7 checks passed
@ReneHezser ReneHezser deleted the spec-output-parameter-decorators branch June 17, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Maintainers need to triage still

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants