Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve backport.yml to only check start of line and instructions for team visibility #64915

Merged
merged 1 commit into from
Feb 7, 2022

Conversation

akoeplinger
Copy link
Member

Before the bot reacted to the /backport command anywhere in the comment, now it needs to be at the start of the line.

Additionally, if the check for repo collaborator fails print a link to the Microsoft team on GitHub to change visibility (most common failure case).

… team visibility

Before the bot reacted to the /backport command anywhere in the comment, now it needs to be at the start of the line.

Additionally, if the check for repo collaborator fails print a link to the Microsoft team on GitHub to change visibility (most common failure case).
@ghost
Copy link

ghost commented Feb 7, 2022

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Before the bot reacted to the /backport command anywhere in the comment, now it needs to be at the start of the line.

Additionally, if the check for repo collaborator fails print a link to the Microsoft team on GitHub to change visibility (most common failure case).

Author: akoeplinger
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@akoeplinger
Copy link
Member Author

Merging since this doesn't affect CI.

@akoeplinger akoeplinger merged commit a512284 into dotnet:main Feb 7, 2022
@akoeplinger akoeplinger deleted the backport-improvement branch February 7, 2022 17:24
@ghost ghost locked as resolved and limited conversation to collaborators Mar 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants