Azure / Azure/azure-rest-api-specs

[suppressions] Add label SuppressionReviewRequired if any changes to suppressions.yaml

Open
#33,873 0 comments 0 reactions 1 assignee Claimed by @mikeharder View on GitHub
Dominant language
TypeSpec
Stars
3.1k
Forks
5.9k
Avg merge
3d 2h
Merged PRs (30d)
424

Description

- [ ] Any changes to suppressions.yaml should cause label `SuppressionReviewRequired` to be added, which will block the PR until label `Approved-Suppression` is applied.
- [ ] For ARM, the ARM Reviewer will add this label, as they already do for suppressions in readme.md
- [ ] For data-plane, the reviewer is TBD
- [ ] Update text for `SuppressionReviewRequired` in NextStepsToMerge, to include suppressions.yaml and perhaps link to more/better docs
- [ ] Update docs for TypeSpecRequirement suppressions, to clarify when the suppression should and should not be applied

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.