Skip to content

[8.0] Bring back yml necessary change types in label checkers - #114167

Merged
carlossanlop merged 1 commit into
dotnet:release/8.0-stagingfrom
carlossanlop:LabelCheckingTypes8
Apr 2, 2025
Merged

[8.0] Bring back yml necessary change types in label checkers#114167
carlossanlop merged 1 commit into
dotnet:release/8.0-stagingfrom
carlossanlop:LabelCheckingTypes8

Conversation

@carlossanlop

Copy link
Copy Markdown
Contributor

Backport of #114165

Tell mode.

There are some on pull_request_target types in the label checking yml files that I removed but they are actually necessary, so the labeler keeps working after for example: pushing new commits, pressing the update the branch, closing and opening the PR.

@carlossanlopcarlossanlop added Servicing-approved Approved for servicing release area-Infrastructure labels Apr 2, 2025
@carlossanlopcarlossanlop self-assigned this Apr 2, 2025
CopilotAI review requested due to automatic review settings April 2, 2025 18:46
@carlossanlop
carlossanlop requested review from a team and jeffhandley as code ownersApril 2, 2025 18:46

CopilotAI 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

This backport restores the necessary trigger event types in the label checker workflows to ensure that labeling actions are processed during various pull request events.

  • Updated .github/workflows/check-service-labels.yml to add "edited" and "synchronize" event types.
  • Updated .github/workflows/check-no-merge-label.yml similarly to include "edited" and "synchronize".

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

FileDescription
.github/workflows/check-service-labels.ymlReintroduces additional trigger events for better label checking behavior in release branches.
.github/workflows/check-no-merge-label.ymlAdds the same trigger events for enhanced label checking on main and release branches.

@carlossanlop
carlossanlop merged commit 886a104 into dotnet:release/8.0-stagingApr 2, 2025
@carlossanlop
carlossanlop deleted the LabelCheckingTypes8 branch April 2, 2025 20:21
@github-actionsgithub-actionsBot locked and limited conversation to collaborators May 3, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-InfrastructureServicing-approvedApproved for servicing release

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants

@carlossanlop@jkoritzinsky