dotnet / dotnet/format

Replace Fabric Bot automation for requiring reviews of PRs that cange the .NET SDK version

Open
#2,052 4 comments 0 reactions 1 assignee Claimed by @jeffhandley View on GitHub
Dominant language
No language data
Stars
1.9k
Forks
173
Avg merge
10d 13h
Merged PRs (30d)
1

Description

This repository has had automation in place that prevents auto-merge of PRs that change the .NET SDK version. That automation was implemented through the Fabric Bot tooling with the following logic:

1. When a pull request is opened
2. If the body of the pull requst contains the string, "Updates sdk.version"
3. And the author of the PR is "dotnet-maestro[bot]"
4. Then issue the action of requiring changes to the pull request, leaving a comment of, "This PR changes the .NET SDK version. Review required before merging."

We need to remove this automation as our migration from Fabric Bot to the Policy Service Resource Management tooling will not support the automated action of requesting changes to a pull request.

After the migration to the new service is completed, we can reintroduce this automation, perhaps with the approach of using the same logic as above, but issuing the action of disabling auto-merge, either through removing the label or using GitHub's built-in auto-merge and disabling that.

/cc @arkalyanms @wtgodbe @mkArtakMSFT @jaredpar

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.