Skip to content

fix: Changed labeler.yml to latest format - #8431

Merged
viirya merged 6 commits into
apache:mainfrom
viirya:fix_labeler
Dec 6, 2023
Merged

fix: Changed labeler.yml to latest format#8431
viirya merged 6 commits into
apache:mainfrom
viirya:fix_labeler

Conversation

@viirya

@viiryaviirya commented Dec 5, 2023•
edited
Loading

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #.

Rationale for this change

Labeler is broken now, e.g. https://github.com/apache/arrow-datafusion/actions/runs/7107345447/job/19348599271?pr=8419.

Run actions/labeler@v5.0.0
The configuration file (path: .github/workflows/dev_pr/labeler.yml) was found locally, reading from the file
Error: Error: found unexpected type for label 'development-process' (should be array of config options)
Error: found unexpected type for label 'development-process' (should be array of config options)

What changes are included in this PR?

Tries to restore labeler CI by following latest format.

Are these changes tested?

Are there any user-facing changes?

@viirya

viirya commented Dec 5, 2023

Copy link
Copy Markdown
MemberAuthor

Hmm, does the change need to be merged to be effective?

@alamb

alamb commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

Somehow is likely related to #8422

alamb
alamb approved these changes Dec 6, 2023
View reviewed changes

@alambalamb 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.

Thank you @viirya

@viirya

viirya commented Dec 6, 2023

Copy link
Copy Markdown
MemberAuthor

Let me merge this first to see if it works. Thanks @alamb

@viirya
viirya merged commit 3ddd5eb into apache:mainDec 6, 2023
@alamb

alamb commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

It seems to have worked!

@viirya

viirya commented Dec 6, 2023

Copy link
Copy Markdown
MemberAuthor

Yea 🎉

appletreeisyellow pushed a commit to appletreeisyellow/datafusion that referenced this pull request Dec 15, 2023
@viirya@appletreeisyellow
fix: Changed labeler.yml to latest format (apache#8431)
d995ae9
* fix: Changed labeler.yml to latest format
* Use all
* More
* More
* Try
* More
@appletreeisyellowappletreeisyellow mentioned this pull request Dec 15, 2023
Closed
@matthewgappmatthewgapp mentioned this pull request Jan 11, 2024
Closed
@logan-keedelogan-keede mentioned this pull request Mar 25, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alambalambalamb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@viirya@alamb