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

Development: Fix automatic pull request labeler CI action #7749

Merged
merged 5 commits into from
Dec 17, 2023

Conversation

b-fein
Copy link
Contributor

@b-fein b-fein commented Dec 8, 2023

Checklist

General

Motivation and Context

GitHub actions labeller introduced a new format for the configuration file.
https://github.com/actions/labeler/releases/tag/v5.0.0

Description

This PR updates the configuration file for the automatic pull request labeler action to the new format introduced with release 5.
It still fails on this branch since the configuration file from the develop branch is always used. I updated the configuration file schema to follow the example in their readme (https://github.com/actions/labeler#basic-examples).

Code Review

  • Code Review 1
  • Code Review 2

@b-fein b-fein changed the title Development: Fix automatic pull request labeler action Development: Fix automatic pull request labeler ci action Dec 11, 2023
@b-fein b-fein marked this pull request as ready for review December 11, 2023 18:29
@b-fein b-fein requested a review from a team as a code owner December 11, 2023 18:29
@b-fein b-fein removed the blocked label Dec 12, 2023
@b-fein b-fein added this to the 6.7.3 milestone Dec 12, 2023
Copy link
Contributor

@Strohgelaender Strohgelaender left a comment

Choose a reason for hiding this comment

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

pull_request_target uses the configuration from the base branch. I used #7779 to test the configuration and it seems to work.

Copy link

@Ceiridge Ceiridge left a comment

Choose a reason for hiding this comment

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

The new changes follow the recommended syntax of the labeler, which looks fully correct: https://github.com/actions/labeler?tab=readme-ov-file#match-object
Therefore approved 👍

@krusche krusche changed the title Development: Fix automatic pull request labeler ci action Development: Fix automatic pull request labeler CI action Dec 17, 2023
@krusche krusche merged commit 60e143d into develop Dec 17, 2023
22 of 23 checks passed
@krusche krusche deleted the dev-fixx-pr-labeller branch December 17, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants